ci: fix python-publish workflow and update CI Python matrix#4
Merged
Conversation
Agent-Id: agent-aa451ee5-71a4-4615-b875-078b204e672c
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two CI/packaging fixes and one packaging-metadata bump.
Why
The
python-publishworkflow has been failing on every push tomain(most recent failure: run 25980222917 from the merge of #3). Thepublish-to-pypijob is correctly gated on tag pushes, butpublish-to-testpypihas no such guard, so every non-tag push tries to republish the existing version to TestPyPI and gets a 400.While I was in the workflows, I also bumped the
tests.ymlmatrix off Python 3.8 (EOL 2024-10-31) and 3.9 (EOL 2025-10-31).Changes
.github/workflows/python-publish.yml: addif: startsWith(github.ref, 'refs/tags/')to thepublish-to-testpypijob, mirroring the existing guard onpublish-to-pypi. Thebuildjob stays unguarded so it still validates the package builds on every push as a CI signal — only the publish jobs get skipped on non-tag pushes..github/workflows/tests.yml: matrix changed from['3.8', '3.9', '3.10', '3.11']to['3.10', '3.11', '3.12', '3.13'].pyproject.toml:requires-pythonbumped from">=3.6.0"to">=3.10"to match the tested matrix floor. This is the only packaging-visible change — anyone on Python 3.6–3.9 will get a resolver error on the next release; both 3.8 and 3.9 are already EOL.No runtime code in
gapmm2/was touched. Action versions (checkout@v4,setup-python@v5,upload/download-artifact@v4,codecov-action@v5,pypa/gh-action-pypi-publish@release/v1) were already current and were not changed.Verification
pytest tests/ -q→ 16 passed in 0.06s (Python 3.11.15)git diff --stat: 3 files changed, 3 insertions(+), 2 deletions(-)The real proof for the publish-workflow fix comes after merge: the next non-tag push to
mainshould show the publish jobs as skipped rather than failed in the Actions tab.Related
While investigating, I confirmed issue #1 (
AssertionError: Introns cannot be less than zero) has been resolved sincev23.11.3via the try/except wrappers ingapmm2/align.py. That issue can be closed separately — not part of this PR.Pull Request opened by Augment Code with guidance from the PR author