chore(ci): pin setup-uv v5 to its underlying commit - #3560
Conversation
jbeckwith-oai
left a comment
There was a problem hiding this comment.
Thanks @MGPOCKY for catching this and for the focused change. I verified that e58605a9b6da7c637471fab8847a5e5a6b8df081 is the annotated v5 tag object and that d4b2f3b6ecc6e67c4457f6d3e41ec42d3d0fcb86 is the commit it points to. Pinning the underlying full commit SHA is the conventional form recommended by GitHub, so the code change itself looks correct.
One factual correction for the PR description: GitHub Actions does accept and currently execute the annotated-tag object SHA—the latest main workflow runs successfully download and run that exact ref. The Commits API returning 422 does not imply that the Actions resolver rejects it. Please reframe the description as normalizing the pin to the underlying commit SHA, rather than fixing an unusable workflow reference.
The PR's workflow is also currently action_required and spawned no jobs, so CI should be allowed to run before merge. I found no code-level issues with the one-line change.
Automated Release PR --- ## 2.52.1 (2026-07-31) Full Changelog: [v2.52.0...v2.52.1](v2.52.0...v2.52.1) ### Chores * **ci:** pin setup-uv v5 to its underlying commit ([#3560](#3560)) ([cbdc98b](cbdc98b)) --- This pull request is managed by Stainless's [GitHub App](https://github.com/apps/stainless-app). The [semver version number](https://semver.org/#semantic-versioning-specification-semver) is based on included [commit messages](https://www.conventionalcommits.org/en/v1.0.0/). Alternatively, you can manually set the version number in the title of this pull request. For a better experience, it is recommended to use either rebase-merge or squash-merge when merging this pull request. 🔗 Stainless [website](https://www.stainlessapi.com) 📚 Read the [docs](https://app.stainlessapi.com/docs) 🙋 [Reach out](mailto:support@stainlessapi.com) for help or questions Co-authored-by: stainless-app[bot] <142633134+stainless-app[bot]@users.noreply.github.com>
Problem
These workflow
uses:pins are invalid as written. Each item below shows the current value, why it is wrong, and the correct ref that must be used instead.1.
.github/workflows/detect-breaking-changes.ymlCurrent (invalid):
Why this is wrong:
e58605a9b6da7c637471fab8847a5e5a6b8df081is not a commit SHA. It is the Git object SHA of an annotated tag (v5).uses:pins (Commits API returnsNo commit found for SHA).v5points to:d4b2f3b6ecc6e67c4457f6d3e41ec42d3d0fcb86.Correct pin:
Test plan
uses:ref resolves as a commit via the GitHub Commits API