ci: enable CodeRabbit auto-review on PRs based on playground - #2454
ci: enable CodeRabbit auto-review on PRs based on playground#2454dcccrypto wants to merge 1 commit into
playground#2454Conversation
CodeRabbit auto-reviews only PRs whose base branch is the repo default (`main`); `reviews.auto_review.base_branches` defaults to an empty list. This repo does almost all of its work against `playground` — 13 of the last 15 merged PRs — so most PRs were getting "Review skipped: auto reviews are disabled on base/target branches other than the default branch" and no automated review at all. Found this while chasing review on #2451/#2452/#2453, all three of which were skipped for exactly this reason. Config key verified against docs.coderabbit.ai/reference/configuration: `reviews.auto_review.base_branches`, a list of regex patterns matched on the base branch name. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 gap
CodeRabbit auto-reviews only PRs whose base branch is the repository default (
main).reviews.auto_review.base_branchesdefaults to an empty list, and this repo has no.coderabbit.yamlat all.But the repo does almost all of its work against
playground:So in practice the large majority of PRs in this repo receive no automated review. Every one gets:
I found this while chasing review on #2451 / #2452 / #2453 — all three were skipped for exactly this reason, which is why they'd sat with zero review signal.
The change
One new file, four lines of config:
Key verified against the configuration reference:
reviews.auto_review.base_branches, a list of regex patterns matched against the PR's base branch name.mainis listed explicitly rather than relying on the implicit default, so the effective set is obvious from the file.One thing I could not verify — please confirm on merge
The docs do not state which branch CodeRabbit reads
.coderabbit.yamlfrom (PR base, head, or repo default). I checked the configuration reference and searched the docs; it's simply not specified.That matters here because the file is landing on
playground, not the default branch:playground→ PR flow.mainbefore it takes effect.Recommend porting to
mainas well — it's harmless in either case, and it removes the ambiguity. Easy to confirm after merge: open any throwaway PR againstplaygroundand check whether CodeRabbit reviews it instead of skipping.I did not want to quietly assume one behaviour and have this look like it worked when it hadn't.
Immediate unblock (already done)
@coderabbitai reviewposted manually on #2451, #2452 and #2453 — that path works regardless of this config, so those three have review running now.🤖 Generated with Claude Code