Skip to content

docs(ci): note the check-approval gate on release PRs - #23

Merged
GSTJ merged 1 commit into
masterfrom
gstj/document-approval-gate
Jul 27, 2026
Merged

docs(ci): note the check-approval gate on release PRs#23
GSTJ merged 1 commit into
masterfrom
gstj/document-approval-gate

Conversation

@GSTJ

@GSTJ GSTJ commented Jul 27, 2026

Copy link
Copy Markdown
Owner

A comment in release.yml for the one thing that needed hands during the v1.0.11 release.

npm, the tag and the GitHub release all completed on their own. The bump PR then sat with no checks reported, because GitHub held both runs as action_required: github-actions[bot] had no merged PR in this repo yet and the fork-PR approval policy was first_time_contributors. Auto-merge waits on checks that never arrive, so the PR just sat there.

The policy is now first_time_contributors_new_to_github and the bot has a merged PR from #22, so this shouldn't recur. Worth a comment regardless: the symptom reads as a hung release, not as something waiting on a click, and the fix is one API call.

v1.0.11 published and tagged fine, then its bump PR sat with no checks
reported: GitHub held both runs as `action_required` because
`github-actions[bot]` had no merged PR in the repo yet and the fork-PR approval
policy was `first_time_contributors`. Auto-merge waits forever on checks that
never report.

Policy is now `first_time_contributors_new_to_github` and the bot has a merged
PR, so it shouldn't come back. Documented anyway with the one command to clear
it, since the symptom looks like a hung release rather than a permission
prompt.
@GSTJ
GSTJ merged commit 3330f5e into master Jul 27, 2026
3 checks passed
@GSTJ
GSTJ deleted the gstj/document-approval-gate branch July 27, 2026 01:13
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