Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
77 changes: 77 additions & 0 deletions .coderabbit.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,77 @@
# SPDX-FileCopyrightText: Copyright (c) 2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
# SPDX-License-Identifier: Apache-2.0

reviews:
profile: chill
high_level_summary: true
high_level_summary_in_walkthrough: true
poem: false
auto_review:
enabled: true
drafts: false
base_branches:
- "^main$"
- "^release/.*"
- "^hotfix/.*"
ignore_usernames: ["rapids-bot", "GPUtester", "nv-automation-bot", "copy-pr-bot"]
tools:
markdownlint:
enabled: true
shellcheck:
enabled: true
gitleaks:
enabled: true
sequence_diagrams: false
collapse_walkthrough: true

# Reduce noise from status messages
request_changes_workflow: false
review_status: false

# Path-specific review instructions
# Note: Detailed C++/CUDA and Python review guidelines are in cpp/agents.md and python/agents.md
path_instructions:
- path: "docs/**/*"
instructions: |
For documentation changes, focus on:
- Accuracy: Verify code examples compile and run correctly
- Completeness: Check if API changes (parameters, return values, errors) are documented
- Clarity: Flag confusing explanations, missing prerequisites, or unclear examples
- Consistency: Version numbers, parameter types, and terminology match code
- Missing docs: If PR changes public APIs without updating docs, flag as HIGH priority
- path: "c/include/cuvs/**/*"
instructions: |
For public C API headers, additionally check:
- Doxygen documentation for all public functions/classes
- API changes flagged for docs/ updates
- Breaking changes require deprecation warnings and migration guide updates
- path: "cpp/include/cuvs/**/*"
instructions: |
For public C++ API headers, additionally check:
- Doxygen documentation for all public functions/classes
- API changes flagged for docs/ updates
- Breaking changes require deprecation warnings and migration guide updates
- path: "notebooks/**/*"
instructions: |
For example notebooks:
- Verify code examples match current API (parameter names, return types)
- Flag outdated imports or deprecated API usage
- path: "ci/**/*"
instructions: |
For CI/build scripts:
- Check for proper conda environment handling
- Verify GPU availability checks before tests
- Check for proper error handling and meaningful error messages
knowledge_base:
opt_out: false
code_guidelines:
filePatterns:
- "cpp/agents.md"
- "python/agents.md"
- "docs/source/contributing.md"
- "docs/source/developer_guide.md"
1 change: 1 addition & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ cpp/scripts/run-cmake-format.sh @rapidsai/cuvs-cmake-codeowners
/.github/ @rapidsai/ci-codeowners
/ci/ @rapidsai/ci-codeowners
/.shellcheckrc @rapidsai/ci-codeowners
/.coderabbit.yaml @rapidsai/ci-codeowners

#packaging code owners
/.pre-commit-config.yaml @rapidsai/packaging-codeowners
Expand Down
14 changes: 14 additions & 0 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ jobs:
files_yaml: |
build_docs:
- '**'
- '!**/*/agents.md'
- '!.coderabbit.yaml'
- '!.devcontainer/**'
- '!.github/CODEOWNERS'
- '!.github/ISSUE_TEMPLATE/**'
Expand All @@ -89,6 +91,8 @@ jobs:
- '!thirdparty/LICENSES/**'
test_cpp:
- '**'
- '!**/*/agents.md'
- '!.coderabbit.yaml'
- '!.devcontainer/**'
- '!.github/CODEOWNERS'
- '!.github/ISSUE_TEMPLATE/**'
Expand Down Expand Up @@ -123,6 +127,8 @@ jobs:
- '!thirdparty/LICENSES/**'
test_java:
- '**'
- '!**/*/agents.md'
- '!.coderabbit.yaml'
- '!.devcontainer/**'
- '!.github/CODEOWNERS'
- '!.github/ISSUE_TEMPLATE/**'
Expand Down Expand Up @@ -156,6 +162,8 @@ jobs:
- '!thirdparty/LICENSES/**'
test_python_conda:
- '**'
- '!**/*/agents.md'
- '!.coderabbit.yaml'
- '!.devcontainer/**'
- '!.github/CODEOWNERS'
- '!.github/ISSUE_TEMPLATE/**'
Expand Down Expand Up @@ -193,6 +201,8 @@ jobs:
- '!thirdparty/LICENSES/**'
test_python_wheels:
- '**'
- '!**/*/agents.md'
- '!.coderabbit.yaml'
- '!.devcontainer/**'
- '!.github/CODEOWNERS'
- '!.github/ISSUE_TEMPLATE/**'
Expand Down Expand Up @@ -235,6 +245,8 @@ jobs:
- '!thirdparty/LICENSES/**'
test_rust:
- '**'
- '!**/*/agents.md'
- '!.coderabbit.yaml'
- '!.devcontainer/**'
- '!.github/CODEOWNERS'
- '!.github/ISSUE_TEMPLATE/**'
Expand Down Expand Up @@ -272,6 +284,8 @@ jobs:
- '!thirdparty/LICENSES/**'
test_go:
- '**'
- '!**/*/agents.md'
- '!.coderabbit.yaml'
- '!.devcontainer/**'
- '!.github/CODEOWNERS'
- '!.github/ISSUE_TEMPLATE/**'
Expand Down
Loading