Skip to content

fix: stop claiming non-release after ALLOW_RELEASE_MERGE bypass#213

Merged
SebTardif merged 1 commit into
mainfrom
fix/guard-release-bypass-message
Jul 24, 2026
Merged

fix: stop claiming non-release after ALLOW_RELEASE_MERGE bypass#213
SebTardif merged 1 commit into
mainfrom
fix/guard-release-bypass-message

Conversation

@SebTardif

Copy link
Copy Markdown
Contributor

Summary

After ALLOW_RELEASE_MERGE=yes on a release PR, the guard printed both the bypass notice and "does not appear to be a release PR". Exit after the bypass message so agents do not misread the outcome.

Test plan

  • Script structure: release + bypass exits 0 without the non-release line
  • CI green

When the strong release guard detects a release PR and ALLOW_RELEASE_MERGE=yes
is set, exit after the bypass message. Do not print the generic "does not
appear to be a release PR" line, which was misleading after a real bypass.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
Copilot AI review requested due to automatic review settings July 24, 2026 13:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@SebTardif
SebTardif enabled auto-merge (squash) July 24, 2026 13:27
@SebTardif
SebTardif merged commit bd17b59 into main Jul 24, 2026
21 checks passed
@SebTardif
SebTardif deleted the fix/guard-release-bypass-message branch July 24, 2026 13:31
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