-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy path.coderabbit.yaml
More file actions
36 lines (34 loc) · 1.69 KB
/
Copy path.coderabbit.yaml
File metadata and controls
36 lines (34 loc) · 1.69 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
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
#
# Versioned CodeRabbit configuration (config-as-code, in preference to hidden
# Org-UI settings). Encodes this repo's documented docstring convention from
# CLAUDE.md: docstrings on PUBLIC functions; comments explain *why*, not *what*;
# test names are self-documenting (a concise one-liner of intent is enough).
#
# Note on the "Docstring Coverage" pre-merge check: its threshold is intentionally
# left at the Org-UI default — this file does not lower or disable it. It only
# guides the docstrings CodeRabbit *generates* so new code matches the house style.
reviews:
profile: assertive
auto_review:
enabled: true
drafts: false
code_generation:
docstrings:
path_instructions:
- path: "backend/app/**/*.py"
instructions: |
Google-style docstrings on public functions and classes, with Args /
Returns / Raises where they add information. Document behaviour and edge
cases; keep comments focused on *why*, not *what*. Private helpers may be
left undocumented when their name and signature are self-evident.
- path: "eval/**/*.py"
instructions: |
Concise docstrings on public functions and classes. For evaluators and
result types, state the honesty-gate behaviour (quotable vs n/a) so the
contract is explicit. Comments explain *why*, not *what*.
- path: "**/test_*.py"
instructions: |
A one-line docstring stating what each test verifies (e.g. the asserted
metric value or invariant). Keep it concise; the test name carries the
rest. Shared fixtures/helpers get a short purpose line.