Skip to content

Add CodeRabbit configuration to scope automated PR reviews - #2804

Merged
karthikvetrivel merged 1 commit into
NVIDIA:mainfrom
karthikvetrivel:add-coderabbit-config
Aug 28, 2026
Merged

Add CodeRabbit configuration to scope automated PR reviews#2804
karthikvetrivel merged 1 commit into
NVIDIA:mainfrom
karthikvetrivel:add-coderabbit-config

Conversation

@karthikvetrivel

@karthikvetrivel karthikvetrivel commented Aug 25, 2026

Copy link
Copy Markdown
Member

Description

I added a .coderabbit.yaml so that CodeRabbit reviews this repo usefully.

Two things about this repo would make a default install noisy.

  • Roughly half of all pull requests are machine generated by Dependabot, Renovate, and the cherry-pick workflow.
  • Vendored and generated files also make up about 90% of tracked files.

The config addresses both:

  • It uses the quiet profile, so only critical and major findings are posted inline.
  • It skips pull requests authored by dependabot[bot] and github-actions[bot]. A backport only replays commits that were already reviewed on main.
  • It filters vendor/, generated deepcopy and clientset code, the CRD copies that make sync-crds produces, and the intentionally malformed fixtures under testdata/.
  • It disables golangci-lint, which CI already runs with our own config, and yamllint, which cannot parse our Helm and Go templates.

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

This change adds no Go code, so there are no new code paths to cover and no Go artifacts to regenerate.

@rajathagasthya rajathagasthya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some suggestions.

Comment thread .coderabbit.yaml Outdated
Comment thread .coderabbit.yaml Outdated
Comment thread .coderabbit.yaml
Comment thread .coderabbit.yaml
Comment thread .coderabbit.yaml
Comment thread .coderabbit.yaml Outdated
Signed-off-by: Karthik Vetrivel <kvetrivel@nvidia.com>
@karthikvetrivel
karthikvetrivel merged commit b98d458 into NVIDIA:main Aug 28, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants