Skip to content

fix: auto-approve automated version bump PRs - #7

Merged
monazhu merged 1 commit into
mainfrom
ci/auto-approve-version-bumps
Aug 12, 2026
Merged

monazhu merged 1 commit into
mainfrom
ci/auto-approve-version-bumps

Conversation

@monazhu

@monazhu monazhu commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #4, replacing the abandoned GitHub App approach (#6). Bump PRs are authored by the PAT owner, who can't approve their own PRs — so the required review blocked every automated bump.

Changes

New auto-approve-version-bump.yaml: approves machine-generated bump PRs as github-actions[bot], satisfying main's 1-review requirement so auto-merge can complete unattended.

Guards — branch protection itself is untouched; every human PR still needs a real review:

  • Fires only on same-repo chore/bump-version-* branches. Fork PRs get a read-only GITHUB_TOKEN and can never approve, settings notwithstanding.
  • Hard-fails unless the diff is exactly the Version: line of DESCRIPTION — a code change smuggled onto a bump branch gets no approval.

Deployment notes

  • Repo Actions setting "Allow GitHub Actions to create and approve pull requests" enabled (default token permissions stay read-only).
  • This PR is fix:-typed, so merging it re-triggers the version bump (PR chore: bump version to 0.1.1 #5 was closed) and exercises the full chain: bump PR → bot approval → auto-merge → v0.1.1 tag.

🤖 Generated with Claude Code

Bump PRs are authored by the PAT owner, who can't approve their own
PRs, so the required review blocked every automated bump. Approve
machine-generated bump PRs as github-actions[bot] instead, guarded to
same-repo chore/bump-version-* branches whose diff is exactly the
DESCRIPTION Version line. Human PRs still require a real review.
@monazhu
monazhu merged commit 693788e into main Aug 12, 2026
9 of 11 checks passed
@monazhu
monazhu deleted the ci/auto-approve-version-bumps branch August 12, 2026 19:18
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