Skip to content

chore(scripts): check PR mergeable state before merging#12219

Merged
jinseopkim0 merged 2 commits intomainfrom
merge_bot
Mar 27, 2026
Merged

chore(scripts): check PR mergeable state before merging#12219
jinseopkim0 merged 2 commits intomainfrom
merge_bot

Conversation

@jinseopkim0
Copy link
Copy Markdown
Contributor

@jinseopkim0 jinseopkim0 commented Mar 26, 2026

The release manager merge bot was previously failing with an error when a PR's combined status was "success" but required checks (like GitHub Actions check runs) or reviews were still missing.

The release manager merge bot was previously failing with a 405 error
when a PR's combined status was "success" but required checks (like
GitHub Actions check runs) or reviews were still missing.
@jinseopkim0 jinseopkim0 requested a review from a team as a code owner March 26, 2026 19:48
@jinseopkim0 jinseopkim0 requested a review from blakeli0 March 26, 2026 19:50
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request enhances the release manager merge bot by incorporating checks for the pull request's mergeable state. It now explicitly handles 'dirty' and 'draft' states as fatal errors and introduces a waiting mechanism when the mergeable state is 'blocked', 'behind', or pending calculation. A review comment suggests refactoring the logic to avoid the use of a goto statement for retrying, recommending more idiomatic Go control flow structures to improve maintainability.

@jinseopkim0 jinseopkim0 enabled auto-merge (squash) March 26, 2026 19:55
@jinseopkim0 jinseopkim0 merged commit 44f9d60 into main Mar 27, 2026
91 checks passed
@jinseopkim0 jinseopkim0 deleted the merge_bot branch March 27, 2026 21:25
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