mirror of
https://github.com/comfyanonymous/ComfyUI.git
synced 2026-03-05 17:27:42 +08:00
chore: tune CodeRabbit config to limit review scope and disable for drafts (#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 #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
This commit is contained in:
parent
ee72752162
commit
b6cb30bab5
@ -1,6 +1,7 @@
|
|||||||
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
|
# yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
|
||||||
language: "en-US"
|
language: "en-US"
|
||||||
early_access: false
|
early_access: false
|
||||||
|
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."
|
||||||
|
|
||||||
reviews:
|
reviews:
|
||||||
profile: "chill"
|
profile: "chill"
|
||||||
@ -35,6 +36,14 @@ reviews:
|
|||||||
- "!**/*.bat"
|
- "!**/*.bat"
|
||||||
|
|
||||||
path_instructions:
|
path_instructions:
|
||||||
|
- path: "**"
|
||||||
|
instructions: |
|
||||||
|
IMPORTANT: Only comment on issues directly introduced by this PR's code changes.
|
||||||
|
Do NOT flag pre-existing issues in code that was merely moved, re-indented,
|
||||||
|
de-indented, or reformatted without logic changes. If code appears in the diff
|
||||||
|
only due to whitespace or structural reformatting (e.g., removing a `with:` block),
|
||||||
|
treat it as unchanged. Contributors should not feel obligated to address
|
||||||
|
pre-existing issues outside the scope of their contribution.
|
||||||
- path: "comfy/**"
|
- path: "comfy/**"
|
||||||
instructions: |
|
instructions: |
|
||||||
Core ML/diffusion engine. Focus on:
|
Core ML/diffusion engine. Focus on:
|
||||||
@ -74,7 +83,11 @@ reviews:
|
|||||||
auto_review:
|
auto_review:
|
||||||
enabled: true
|
enabled: true
|
||||||
auto_incremental_review: true
|
auto_incremental_review: true
|
||||||
drafts: true
|
drafts: false
|
||||||
|
ignore_title_keywords:
|
||||||
|
- "WIP"
|
||||||
|
- "DO NOT REVIEW"
|
||||||
|
- "DO NOT MERGE"
|
||||||
|
|
||||||
finishing_touches:
|
finishing_touches:
|
||||||
docstrings:
|
docstrings:
|
||||||
@ -84,7 +97,7 @@ reviews:
|
|||||||
|
|
||||||
tools:
|
tools:
|
||||||
ruff:
|
ruff:
|
||||||
enabled: true
|
enabled: false
|
||||||
pylint:
|
pylint:
|
||||||
enabled: false
|
enabled: false
|
||||||
flake8:
|
flake8:
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user