Skip to content

ci: pin the publish action to a commit, not a tag object - #6837

Merged
u9g merged 1 commit into
mainfrom
longc/fix-pypi-action-pin
Aug 13, 2026
Merged

ci: pin the publish action to a commit, not a tag object#6837
u9g merged 1 commit into
mainfrom
longc/fix-pypi-action-pin

Conversation

@longcw

@longcw longcw commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Problem

#6836 pinned the publish action to a892a5a, which is the annotated tag object of v1.14.2, not the commit it points to. The 1.6.10 republish then stopped:

Unable to find image 'ghcr.io/pypa/gh-action-pypi-publish:a892a5a61159132606e93a2fa6f4358831b04d26'
docker: Error response from daemon: manifest unknown

gh-action-pypi-publish is a composite action. It builds a Docker action at run time from github.action_ref and pulls ghcr.io/pypa/gh-action-pypi-publish:<that ref>. The pin must therefore be a ref with a published image. A tag object is not one.

The registry agrees:

dc37677b (the commit)   HTTP 200
a892a5a  (the tag)      HTTP 404

Fix

Pin the commit that v1.14.2 points to. It is the same tree #6836 intended, with twine==7.0.0 and packaging==26.2, which accept the Metadata-Version: 2.5 wheels that hatchling 1.32.0 produces.

To confirm a pin of this kind, ask the API for the commit. A tag object gives 422:

gh api repos/pypa/gh-action-pypi-publish/commits/<sha>

#6836 used the sha of the v1.14.2 annotated tag. The action pulls
ghcr.io/pypa/gh-action-pypi-publish tagged by github.action_ref, and no image
is published for a tag object, so the republish failed with `manifest unknown`.

Use the commit v1.14.2 points to. Same tree, same twine 7.0.0.
@longcw
longcw requested a review from a team as a code owner August 13, 2026 03:21

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@u9g
u9g merged commit 6f181b5 into main Aug 13, 2026
26 checks passed
@u9g
u9g deleted the longc/fix-pypi-action-pin branch August 13, 2026 03:24
@longcw

longcw commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants