Skip to content

fix(release): handle GitHub PR number suffix in commit message check#145

Merged
mdelapenya merged 1 commit intomainfrom
fix/release-regex-pr-number
Mar 4, 2026
Merged

fix(release): handle GitHub PR number suffix in commit message check#145
mdelapenya merged 1 commit intomainfrom
fix/release-regex-pr-number

Conversation

@mdelapenya
Copy link
Member

Summary

  • Phase 2 auto-tagging was skipped because the commit message regex didn't account for GitHub's (#NNN) suffix on squash merge titles
  • The squash merge of chore(release): bump module versions #144 produced chore(release): bump module versions (#144) which failed the versions?[[:space:]]*$ pattern
  • Updated regex to accept optional (#NNN) PR number suffix

Test plan

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>
@mdelapenya mdelapenya requested a review from glours March 4, 2026 16:17
@mdelapenya mdelapenya merged commit f7329ac into main Mar 4, 2026
9 checks passed
@mdelapenya mdelapenya deleted the fix/release-regex-pr-number branch March 4, 2026 16:17
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