Skip to content

Commit f7329ac

Browse files
mdelapenyaclaude
andauthored
fix(release): handle GitHub PR number suffix in commit message check (#145)
GitHub appends (#NNN) to squash merge commit titles, which caused the Phase 2 release regex to fail and skip auto-tagging. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 2a5f8f3 commit f7329ac

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ jobs:
162162
# - Squash merge: PR title is the subject line
163163
# - Regular merge: PR title appears in the body
164164
# - Rebase merge: original commit message is preserved
165-
if echo "$COMMIT_MESSAGES" | grep -qE '^chore\((release|[a-z0-9_-]+)\): bump (module )?versions?[[:space:]]*$'; then
165+
if echo "$COMMIT_MESSAGES" | grep -qE '^chore\((release|[a-z0-9_-]+)\): bump (module )?versions?([[:space:]]|\(#[0-9]+\))*$'; then
166166
echo "✅ Push contains a release commit — proceeding with tagging"
167167
echo "is_release=true" >> "$GITHUB_OUTPUT"
168168
else

0 commit comments

Comments
 (0)