Skip to content

Commit ca867a1

Browse files
committed
fix: stop claiming non-release after ALLOW_RELEASE_MERGE bypass
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>
1 parent 2f3d9da commit ca867a1

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

scripts/guard-no-release-merge.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,9 @@ if [[ "$is_release" == "true" ]]; then
7272

7373
if [[ "${ALLOW_RELEASE_MERGE:-}" != "yes" ]]; then
7474
exit 1
75-
else
76-
echo "BYPASS: ALLOW_RELEASE_MERGE=yes detected. Proceeding (user approved)."
7775
fi
76+
echo "BYPASS: ALLOW_RELEASE_MERGE=yes detected. Proceeding (user approved)."
77+
exit 0
7878
fi
7979

8080
echo "Guard OK: PR #$pr does not appear to be a release PR. Safe to consider merge."

0 commit comments

Comments
 (0)