Conversation
Matrix cell names carry their ruby and gemfile, so a branch-protection rule listing them silently stops requiring anything when the matrix changes shape -- it just did (#295) and #249 will change it again. `skipped` passes: the matrix is off on ordinary PRs by design. `cancelled` does not -- a cell killed before it reported carries no verdict.
|
Warning Review limit reachedNext included review available in 16 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
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 |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds an always-evaluated File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Closes the last box on #281.
Matrix cell names carry their ruby and gemfile, so a branch-protection rule that lists them silently stops requiring anything when the matrix changes shape — it just did (#295), and #249 will change it again.
CI Gatedepends on the jobs instead of naming them. Branch protection requires that one check, forever.skippedpasses — the matrix is off on ordinary PRs by design.cancelledfails — a cell killed before it reported carries no verdict.Human step after merge: Settings → Branches → require
CI Gateonmaster.Summary by Sourcery
Add a stable aggregate CI check for branch protection across changing test matrices.
Enhancements:
CI: