fix(ci): accept breaking-change markers in the PR labeler title regex - #6780
Conversation
|
Warning Review limit reached
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 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 configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
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. Comment |
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>
4c234b9 to
71dab13
Compare
|
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
!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]yieldsclaude, 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 integrationnow matches (typerefactor, scopecar); plainfix(ci): ...titles match exactly as before.🤖 Generated with Claude Code