Skip to content

fix(ci): accept breaking-change markers in the PR labeler title regex - #6780

Merged
jamesarich merged 1 commit into
mainfrom
claude/loving-blackburn-9d831c
Aug 20, 2026
Merged

fix(ci): accept breaking-change markers in the PR labeler title regex#6780
jamesarich merged 1 commit into
mainfrom
claude/loving-blackburn-9d831c

Conversation

@jamesarich

Copy link
Copy Markdown
Collaborator

The PR auto-labeler's conventional-commit title regex required the type/scope to be followed immediately by :, so titles carrying the breaking-change marker ! (e.g. refactor(car)!: remove the Android Auto integration, #6779) failed to match and the run ended with "No labels matched".

🐛 Bug Fixes

  • Allow an optional ! before the colon in the labeler's title regex in .github/workflows/pull-request-target.yml (/^(\w+)(?:\(([^)]+)\))?://^(\w+)(?:\(([^)]+)\))?!?:/). Everything else in the script is unchanged.

Not changed, just noting: the branch-prefix fallback also doesn't help claude/-prefixed branches — branch.split(/[\/\-_]/)[0] yields claude, which maps to no label — so on such branches the title regex is the only type/scope signal. Left as-is deliberately.

Testing Performed

Workflow-only change, no Gradle tasks run. Verified the new regex against both title shapes in a JS scratch check: refactor(car)!: remove the Android Auto integration now matches (type refactor, scope car); plain fix(ci): ... titles match exactly as before.

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@jamesarich, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 17 minutes

Limit details: You’ve used all 4 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: bda317b0-de3f-425c-b781-0fe2dfef050f

📥 Commits

Reviewing files that changed from the base of the PR and between fee0b35 and 71dab13.

📒 Files selected for processing (1)
  • .github/workflows/pull-request-target.yml
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added bugfix PR tag repo Repository maintenance labels Aug 19, 2026
@jamesarich
jamesarich marked this pull request as ready for review August 19, 2026 23:53
The labeler's conventional-commit title regex required the type/scope to
be followed immediately by ":", so titles carrying the breaking-change
marker "!" (e.g. "refactor(car)!: ...") failed to match and the PR got
no type/scope labels (observed on #6779). Allow an optional "!" before
the colon.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@jamesarich
jamesarich force-pushed the claude/loving-blackburn-9d831c branch from 4c234b9 to 71dab13 Compare August 19, 2026 23:53
@jamesarich
jamesarich enabled auto-merge August 20, 2026 00:01
@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@jamesarich
jamesarich added this pull request to the merge queue Aug 20, 2026
Merged via the queue into main with commit 2bdc1c8 Aug 20, 2026
25 of 27 checks passed
@jamesarich
jamesarich deleted the claude/loving-blackburn-9d831c branch August 20, 2026 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix PR tag repo Repository maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant