Skip to content

Commit a939d01

Browse files
Merge pull request #992 from python-wheel-build/chore/add-coderabbit-config
chore: add CodeRabbit configuration for AI-assisted code review
2 parents 472e16a + ac7798f commit a939d01

1 file changed

Lines changed: 135 additions & 0 deletions

File tree

.coderabbit.yaml

Lines changed: 135 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,135 @@
1+
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
2+
# CodeRabbit Configuration for python-wheel-build/fromager
3+
4+
language: "en-US"
5+
early_access: false
6+
inheritance: true
7+
8+
tone_instructions: >-
9+
Be direct and concise. Provide code suggestions when flagging issues.
10+
Skip comments already handled by ruff, mypy, or pre-commit. Only comment
11+
on specific bugs, security issues, or measurable improvements.
12+
13+
reviews:
14+
profile: chill
15+
16+
request_changes_workflow: false
17+
18+
high_level_summary: false
19+
high_level_summary_in_walkthrough: true
20+
poem: false
21+
review_status: true
22+
collapse_walkthrough: true
23+
sequence_diagrams: false
24+
changed_files_summary: false
25+
suggested_labels: false
26+
suggested_reviewers: false
27+
related_issues: false
28+
related_prs: false
29+
30+
commit_status: true
31+
fail_commit_status: false
32+
33+
finishing_touches:
34+
docstrings:
35+
enabled: false
36+
unit_tests:
37+
enabled: false
38+
39+
pre_merge_checks:
40+
description:
41+
mode: warning
42+
docstrings:
43+
mode: off
44+
45+
tools:
46+
# Disable tools that fromager already runs via hatch/pre-commit
47+
ruff:
48+
enabled: false
49+
flake8:
50+
enabled: false
51+
pylint:
52+
enabled: false
53+
markdownlint:
54+
enabled: false
55+
56+
# Not applicable (no JS/TS/Go in fromager)
57+
biome:
58+
enabled: false
59+
eslint:
60+
enabled: false
61+
golangci-lint:
62+
enabled: false
63+
64+
# Enable security scanning tools
65+
gitleaks:
66+
enabled: true
67+
semgrep:
68+
enabled: true
69+
shellcheck:
70+
enabled: true
71+
yamllint:
72+
enabled: true
73+
actionlint:
74+
enabled: true
75+
checkov:
76+
enabled: true
77+
78+
path_instructions:
79+
- path: "**"
80+
instructions: >-
81+
Focus on major issues impacting performance, readability,
82+
maintainability and security. Avoid nitpicks and verbosity.
83+
84+
- path: "tests/**"
85+
instructions: >-
86+
Verify test actually tests the intended behavior. Check for missing
87+
edge cases. Flag overly brittle mocks. Skip all style comments —
88+
ruff handles that.
89+
90+
- path: "e2e/**/*.sh"
91+
instructions: >-
92+
Check for proper cleanup and teardown (trap handlers). Look for race
93+
conditions in server startup/shutdown. Ensure set -euo pipefail is
94+
used. Verify variables are quoted to prevent word splitting.
95+
96+
- path: ".github/workflows/**"
97+
instructions: >-
98+
Check that actions are pinned by SHA, not tag (supply chain security).
99+
No secret interpolation in run: blocks. Least-privilege permissions
100+
per job.
101+
102+
- path: "**/*.md"
103+
instructions: >-
104+
Only comment on factual errors or broken links. Skip all formatting
105+
and style suggestions.
106+
107+
path_filters:
108+
- "!**/*.pyc"
109+
- "!**/__pycache__/**"
110+
- "!.venv/**"
111+
- "!venv/**"
112+
- "!htmlcov/**"
113+
- "!.pytest_cache/**"
114+
- "!.ruff_cache/**"
115+
- "!*.egg-info/**"
116+
- "!build/**"
117+
- "!dist/**"
118+
- "!.mypy_cache/**"
119+
- "!*.log"
120+
- "!*.tmp"
121+
122+
auto_review:
123+
enabled: true
124+
drafts: false
125+
ignore_title_keywords:
126+
- "WIP"
127+
- "DO NOT MERGE"
128+
labels:
129+
- "!work-in-progress"
130+
131+
knowledge_base:
132+
learnings:
133+
scope: local
134+
issues:
135+
scope: local

0 commit comments

Comments
 (0)