Skip to content

fix: use PAT for release-please so release PR triggers required check#21

Merged
roleme merged 1 commit into
mainfrom
fix/release-please-pat
Jun 26, 2026
Merged

fix: use PAT for release-please so release PR triggers required check#21
roleme merged 1 commit into
mainfrom
fix/release-please-pat

Conversation

@roleme

@roleme roleme commented Jun 26, 2026

Copy link
Copy Markdown
Owner

Why

Follow-up to #19. The first release-please run created the release PR with the default GITHUB_TOKEN. By GitHub's loop-prevention design, a PR opened by GITHUB_TOKEN does not trigger pull_request workflows — so the required zizmor check never ran on the release PR and it sat permanently BLOCKED, unable to auto-merge.

Fix

Pass RELEASE_PLEASE_TOKEN (fine-grained PAT scoped to this repo: contents + pull-requests write) to:

  • the release-please-action token: input — so its release PR triggers zizmor
  • the auto-merge gh step — consistent identity with pull-request write

Now the release PR runs zizmor, goes green, and auto-merges for patch/minor bumps; major bumps stay open for manual approval.

Stale release PR #20 (opened by GITHUB_TOKEN, no checks) was closed; after this merges, release-please opens a fresh release PR via the PAT.

🤖 Generated with Claude Code

The default GITHUB_TOKEN cannot trigger pull_request workflows (GitHub
loop-prevention), so the release PR never got a `zizmor` run and stayed
permanently BLOCKED under branch protection. Pass RELEASE_PLEASE_TOKEN (a
fine-grained PAT scoped to this repo: contents + pull-requests write) to the
action and the auto-merge step so the release PR triggers zizmor and can
auto-merge on green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@roleme roleme merged commit f7aa8b5 into main Jun 26, 2026
1 check passed
@roleme roleme deleted the fix/release-please-pat branch June 26, 2026 11:35
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.

1 participant