Commit a8d9f13
ci(sync): drop auto-merge step; bot PRs await codeowner review
Auto-merge was failing with `Auto merge is not allowed for this
repository (enablePullRequestAutoMerge)` even after enabling
`allow_auto_merge` at the repo level. Root cause: branch protection
requires codeowner review, the github-actions[bot] cannot bypass
that requirement (GitHub's API silently rejects the
`github-actions` slug from `bypass_pull_request_allowances.apps`),
and there are no other code owners.
Two paths from here, both deferred to a follow-up:
1. Provision a GitHub App owned by amacsmith with Contents:Write +
PullRequests:Write permissions, install on the repo, add its
slug to the bypass allowlist. The sync workflow uses
actions/create-github-app-token to mint a token and the
enable-pull-request-automerge step uses it. Most secure.
2. Mint a fine-grained PAT for amacsmith with the same scopes,
store as secrets.BOT_PAT, use directly. Easier, less granular.
Until either lands, the daily sync workflow still opens / updates a
bot/sync PR; a codeowner merges it manually. Branch-protection
remains intact (codeowner review required) — Scorecard
Branch-Protection score stays at 7.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>1 parent 777ee45 commit a8d9f13
1 file changed
Lines changed: 8 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
157 | 157 | | |
158 | 158 | | |
159 | 159 | | |
160 | | - | |
161 | | - | |
162 | | - | |
163 | | - | |
164 | | - | |
165 | | - | |
166 | | - | |
167 | | - | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
168 | 168 | | |
169 | 169 | | |
170 | 170 | | |
| |||
0 commit comments