forked from rapidsai/cudf
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy path.coderabbit.yaml
More file actions
72 lines (65 loc) · 2.49 KB
/
.coderabbit.yaml
File metadata and controls
72 lines (65 loc) · 2.49 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
# SPDX-FileCopyrightText: Copyright (c) 2025-2026 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/REVIEW_GUIDELINES.md and python/REVIEW_GUIDELINES.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: "java/**/*"
instructions: |
For Java bindings:
- Verify JNI resource cleanup (prevent native memory leaks)
- Check that Java API matches the underlying C++ API behavior
- Ensure proper exception handling across JNI boundary
- Verify AutoCloseable/try-with-resources patterns for GPU resources
- 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/REVIEW_GUIDELINES.md"
- "python/REVIEW_GUIDELINES.md"
- "cpp/doxygen/developer_guide/DEVELOPER_GUIDE.md"
- "cpp/doxygen/developer_guide/TESTING.md"
- "CONTRIBUTING.md"