fix: release-please auto-merge + tag-move work without a checkout#23
Merged
Conversation
The auto-merge step ran `gh pr view` but the job has no checkout, so gh had
no repo context and the call failed ("not a git repository"), leaving every
release PR unmerged. Parse the number+title directly from steps.release
.outputs.prs (already JSON) and pass --repo to `gh pr merge`.
The major-tag-move step likewise relied on local git in a checkout-less job;
switch it to the REST API (create-or-update ref) using the PAT.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Why
Follow-up to #21. The release PR (#22) was created correctly via the PAT and zizmor went green, but it did not auto-merge.
Root cause: the "Enable auto-merge" step called
gh pr view <n>to read the title, but this job has noactions/checkout, soghhad no repo to infer and failed withnot a git repository→ version parse empty → every PR fell through to "leaving for manual review".The major-tag-move step had the same latent bug (it used local
git tag/git pushin a checkout-less job) and would have failed the first time a release was actually cut.Fix
number+titlestraight fromsteps.release.outputs.prs(already a JSON array — nogh pr viewneeded) and pass--repotogh pr merge.refs/tags/vNusing the PAT.After this merges, release-please re-runs and PR #22 should auto-merge once its zizmor check is green.
🤖 Generated with Claude Code