Skip to content

Enables automatic CodeRabbit reviews for pull requests targeting main.Description - #2958

Open
jpascucci-nv wants to merge 1 commit into
NVIDIA:mainfrom
jpascucci-nv:issue2137
Open

jpascucci-nv wants to merge 1 commit into
NVIDIA:mainfrom
jpascucci-nv:issue2137

Conversation

@jpascucci-nv

@jpascucci-nv jpascucci-nv commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Addresses #2137 rollout steps 1 and 2.

Description

The new .coderabbit.yaml is based on CCCL’s low-noise configuration and adapts its review guidance to cuda-python’s packages and workflows. It:

  • Uses the chill review profile with brief, severity-prefixed comments.
  • Disables walkthrough/status noise, finishing-touch suggestions, and pre-merge checks.
  • Skips drafts and PRs authored by repository automation bots.
  • Enables Gitleaks, markdownlint, and ShellCheck.
  • Adds focused review instructions for cuda-bindings, cuda-core, cuda-pathfinder, the cuda-python metapackage, benchmarks, CI, GitHub automation, and Cython sources.
  • Uses the repository’s existing AGENTS.md, contribution guides, package documentation, and release-note indexes as review context.

Automatic reviews are intentionally limited to main. CodeRabbit and Greptile will initially run together so their signal, noise, and overlap can be evaluated on real PRs before expanding CodeRabbit to backport branches or changing either reviewer’s configuration.

Testing

  • Parsed .coderabbit.yaml successfully.
  • Ran YAML linting.
  • Verified the SPDX header.
  • Verified every configured knowledge-base file exists.
  • Verified auto-review is enabled only for main.
  • Ran git diff --check.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

Enables automatic CodeRabbit reviews for pull requests targeting `main`.

The new `.coderabbit.yaml` is based on CCCL’s low-noise configuration and adapts its review guidance to cuda-python’s packages and workflows. It:

- Uses the `chill` review profile with brief, severity-prefixed comments.
- Disables walkthrough/status noise, finishing-touch suggestions, and pre-merge checks.
- Skips drafts and PRs authored by repository automation bots.
- Enables Gitleaks, markdownlint, and ShellCheck.
- Adds focused review instructions for cuda-bindings, cuda-core, cuda-pathfinder, the cuda-python metapackage, benchmarks, CI, GitHub automation, and Cython sources.
- Uses the repository’s existing `AGENTS.md`, contribution guides, package documentation, and release-note indexes as review context.

Automatic reviews are intentionally limited to `main`. CodeRabbit and Greptile will initially run together so their signal, noise, and overlap can be evaluated on real PRs before expanding CodeRabbit to backport branches or changing either reviewer’s configuration.

Testing

- Parsed `.coderabbit.yaml` successfully.
- Ran YAML linting.
- Verified the SPDX header.
- Verified every configured knowledge-base file exists.
- Verified auto-review is enabled only for `main`.
- Ran `git diff --check`.
@jpascucci-nv jpascucci-nv added this to the cuda.core 1.3.0 milestone Sep 28, 2026
@jpascucci-nv jpascucci-nv self-assigned this Sep 28, 2026
@jpascucci-nv jpascucci-nv added the CI/CD CI/CD infrastructure label Sep 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant