Skip to content

Commit b6cb30b

Browse files
chore: tune CodeRabbit config to limit review scope and disable for drafts (Comfy-Org#12567)
* chore: tune CodeRabbit config to limit review scope and disable for drafts - Add tone_instructions to focus only on newly introduced issues - Add global path_instructions entry to ignore pre-existing issues in moved/reformatted code - Disable draft PR reviews (drafts: false) and add WIP title keywords - Disable ruff tool to prevent linter-based outside-diff-range comments Addresses feedback from maintainers about CodeRabbit flagging pre-existing issues in code that was merely moved or de-indented (e.g., PR Comfy-Org#12557), which can discourage community contributions and cause scope creep. Amp-Thread-ID: https://ampcode.com/threads/T-019c82de-0481-7253-ad42-20cb595bb1ba * chore: add 'DO NOT MERGE' to ignore_title_keywords Amp-Thread-ID: https://ampcode.com/threads/T-019c82de-0481-7253-ad42-20cb595bb1ba
1 parent ee72752 commit b6cb30b

1 file changed

Lines changed: 15 additions & 2 deletions

File tree

.coderabbit.yaml

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
22
language: "en-US"
33
early_access: false
4+
tone_instructions: "Only comment on issues introduced by this PR's changes. Do not flag pre-existing problems in moved, re-indented, or reformatted code."
45

56
reviews:
67
profile: "chill"
@@ -35,6 +36,14 @@ reviews:
3536
- "!**/*.bat"
3637

3738
path_instructions:
39+
- path: "**"
40+
instructions: |
41+
IMPORTANT: Only comment on issues directly introduced by this PR's code changes.
42+
Do NOT flag pre-existing issues in code that was merely moved, re-indented,
43+
de-indented, or reformatted without logic changes. If code appears in the diff
44+
only due to whitespace or structural reformatting (e.g., removing a `with:` block),
45+
treat it as unchanged. Contributors should not feel obligated to address
46+
pre-existing issues outside the scope of their contribution.
3847
- path: "comfy/**"
3948
instructions: |
4049
Core ML/diffusion engine. Focus on:
@@ -74,7 +83,11 @@ reviews:
7483
auto_review:
7584
enabled: true
7685
auto_incremental_review: true
77-
drafts: true
86+
drafts: false
87+
ignore_title_keywords:
88+
- "WIP"
89+
- "DO NOT REVIEW"
90+
- "DO NOT MERGE"
7891

7992
finishing_touches:
8093
docstrings:
@@ -84,7 +97,7 @@ reviews:
8497

8598
tools:
8699
ruff:
87-
enabled: true
100+
enabled: false
88101
pylint:
89102
enabled: false
90103
flake8:

0 commit comments

Comments
 (0)