Skip to content

fix: author version-bump PRs as GitHub App - #6

Closed
monazhu wants to merge 1 commit into
mainfrom
ci/version-bump-app-token
Closed

monazhu wants to merge 1 commit into
mainfrom
ci/version-bump-app-token

Conversation

@monazhu

@monazhu monazhu commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #4. The version-bump workflow authenticated with a personal fine-grained PAT, which made every bump PR authored by that PAT's owner — who then couldn't approve their own bump PRs (main requires one approving review). Switch to a GitHub App (c3d-version-bump) so bump PRs are bot-authored and any team member can approve.

Changes

  • version-bump.yaml: mint a short-lived installation token via actions/create-github-app-token@v2 (app id from VERSION_BUMP_APP_ID variable, private key from VERSION_BUMP_APP_KEY secret) and use it for checkout, push, and gh PR operations. Replaces the VERSION_BUMP_TOKEN secret.

Deployment notes

  • Requires the c3d-version-bump app installed on this repo with Contents R/W + Pull requests R/W, plus the VERSION_BUMP_APP_ID variable and VERSION_BUMP_APP_KEY secret.
  • VERSION_BUMP_TOKEN secret can be deleted after this merges.
  • PR chore: bump version to 0.1.1 #5 (bump to 0.1.1) was closed; merging this PR (fix:-typed) re-triggers the bump so the new app-authored flow gets tested end to end.

🤖 Generated with Claude Code

Bump PRs were authored by the personal PAT's owner, who then couldn't
approve their own PRs (main requires one review). Mint a short-lived
app token instead so bump PRs are bot-authored and anyone on the team
can approve.
@monazhu

monazhu commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

Superseded — going with a guarded auto-approve workflow instead of a GitHub App (org-admin access not available); see follow-up PR.

@monazhu monazhu closed this Aug 12, 2026
@monazhu
monazhu deleted the ci/version-bump-app-token branch August 12, 2026 18:44
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