mirror of
https://github.com/actualbudget/actual.git
synced 2026-03-11 20:44:32 -05:00
771e5072cb
* Add custom checks to CodeRabbit configuration - Add settings check to evaluate new setting toggles against core design principles - Add linting check to prevent oxlint-disable lines - Add typecheck check to prevent @ts-strict-ignore in new components/utilities * Add release notes for custom checks in CodeRabbit configuration * Refactor custom checks in CodeRabbit configuration for clarity and consistency - Consolidate custom checks under a single list format for better readability - Maintain existing checks for settings, linting, and typecheck with unchanged instructions
36 lines
1.9 KiB
YAML
36 lines
1.9 KiB
YAML
issue_enrichment:
|
|
auto_enrich:
|
|
enabled: false
|
|
reviews:
|
|
request_changes_workflow: true
|
|
review_status: true
|
|
high_level_summary: false
|
|
finishing_touches:
|
|
docstrings:
|
|
enabled: false
|
|
pre_merge_checks:
|
|
docstrings:
|
|
enabled: false
|
|
custom_checks:
|
|
- mode: error
|
|
name: 'settings'
|
|
instructions: 'Every addition of a new setting toggle must be thoroughly evaluated against the core design principles of Actual. The settings screen is reserved for essential and foundational options only — do not introduce settings for minor UI adjustments such as sizes, paddings, colors, or margins. Prioritize preserving a simple and uncluttered user experience. Users proposing new settings must confirm in a reply to the Coderabbit comment that they have reviewed and ensured alignment with these principles. Excessive or granular UI options increase code complexity and risk confusing users, and are generally not permitted.'
|
|
- mode: error
|
|
name: 'linting'
|
|
instructions: 'Do not allow any oxlint-disable lines.'
|
|
- mode: error
|
|
name: 'typecheck'
|
|
instructions: 'Do not allow creating new components or utilities with the @ts-strict-ignore comment.'
|
|
labeling_instructions:
|
|
- label: 'suspect ai generated'
|
|
instructions: 'This issue or PR is suspected to be generated by AI.'
|
|
- label: 'API'
|
|
instructions: 'This issue or PR updates the API in `packages/api`.'
|
|
- label: 'documentation'
|
|
instructions: 'This issue updates the documentation in `packages/docs`.'
|
|
- label: 'contains DB migrations'
|
|
instructions: 'This issue or PR contains DB migrations in `packages/loot-core/migrations`.'
|
|
- label: 'size small'
|
|
instructions: 'This issue or PR is a small change (less than 50 lines of code) that is expected to have a small impact on the codebase.'
|
|
auto_apply_labels: true
|