Skip to content

ci: enable CodeRabbit auto-review on draft PRs#690

Merged
mchmarny merged 2 commits into
NVIDIA:mainfrom
yuanchen8911:coderabbit-review-drafts
Apr 27, 2026
Merged

ci: enable CodeRabbit auto-review on draft PRs#690
mchmarny merged 2 commits into
NVIDIA:mainfrom
yuanchen8911:coderabbit-review-drafts

Conversation

@yuanchen8911

Copy link
Copy Markdown
Contributor

Summary

  • Flip reviews.auto_review.drafts from false to true in .coderabbit.yaml.
  • CodeRabbit will now run auto-review on draft PRs in addition to ready-for-review PRs.

Motivation / Context

Authors who open a PR as draft (to share work-in-progress, run CI, or invite early collaboration) currently get no CodeRabbit feedback until the PR is marked ready-for-review. Enabling drafts: true provides early CodeRabbit feedback on draft PRs so issues can be caught and fixed before the PR is promoted to ready-for-review.

Per CodeRabbit configuration docs, the .coderabbit.yaml on the feature branch is what governs that PR's review, and reviews are triggered on new PRs or incremental commits. Existing draft PRs opened before this change may need a new push or an @coderabbitai review comment to pick up the new behavior; new draft PRs (and any draft PR rebased onto main after this lands) will be auto-reviewed.

Fixes: N/A
Related: N/A

Type of Change

  • CI / build configuration

Components Affected

  • .coderabbit.yaml

Implementation Notes

Single-line change: drafts: falsedrafts: true.

Testing

  • yamllint .coderabbit.yaml — clean.
  • Full make qualify skipped: this is a CI/config-only change to .coderabbit.yaml that does not affect Go code, tests, e2e, sidebar docs, or tools/check-agents-sync inputs. Per the doc-only / infra-only verification policy, the smallest applicable check (yamllint) was run.

Risk Assessment

Low. Reversible by flipping the flag back. CodeRabbit will simply produce additional reviews on draft PRs; no production behavior changes.

Checklist

  • Code follows project style
  • Self-reviewed the change
  • Updated relevant documentation (none needed — internal CI tool config)
  • Verified no secrets or sensitive info added

Flip reviews.auto_review.drafts to true so CodeRabbit reviews draft PRs
in addition to ready-for-review PRs.
@coderabbitai

coderabbitai Bot commented Apr 26, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: b9c71b1d-90a0-4807-b147-fa1126fc97d9

📥 Commits

Reviewing files that changed from the base of the PR and between 3c9f6ec and 8d751d9.

📒 Files selected for processing (1)
  • .coderabbit.yaml

📝 Walkthrough

Walkthrough

A configuration setting in .coderabbit.yaml is modified to enable automatic reviews for draft pull requests. The reviews.auto_review.drafts setting is changed from false to true. This is the sole modification in the pull request, affecting only one line of configuration without any code logic or public entity changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: enabling CodeRabbit auto-review on draft PRs, which matches the single-line configuration change in .coderabbit.yaml.
Description check ✅ Passed The description is comprehensive and directly related to the changeset, explaining the motivation, implementation details, testing, and risk assessment for the configuration change.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@mchmarny mchmarny enabled auto-merge (squash) April 27, 2026 15:00
@mchmarny mchmarny merged commit fc0daca into NVIDIA:main Apr 27, 2026
28 checks passed
lockwobr pushed a commit that referenced this pull request Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants