Skip to content

Adjust coderabbit config to better fit our development style#8035

Merged
lagru merged 6 commits intoscikit-image:mainfrom
stefanv:coderabbit-tweaks
Jan 27, 2026
Merged

Adjust coderabbit config to better fit our development style#8035
lagru merged 6 commits intoscikit-image:mainfrom
stefanv:coderabbit-tweaks

Conversation

@stefanv
Copy link
Copy Markdown
Member

@stefanv stefanv commented Jan 22, 2026

  • Mutually exclusive labels
  • Only respond when asked to
  • Learn the contributor guidelines

@coderabbitai coderabbitai bot added the 🤖 type: Infrastructure CI, packaging, tools and automation label Jan 22, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 22, 2026

📝 Walkthrough

Walkthrough

Updated .coderabbit.yaml: added top-level flags and pre-merge title check, extended knowledge_base code-guideline patterns, and appended "Mutually exclusive." to multiple label descriptions. No public API or signature changes.

Changes

Cohort / File(s) Summary
Configuration additions
\.coderabbit.yaml
Added request_changes_workflow: false, enable_prompt_for_ai_agents: false, chat.auto_reply: false, and auto_merge.pre_merge.pre_merge_checks.title requirement.
Knowledge base patterns
\.coderabbit.yaml
Added knowledge_base.code_guidelines.filePatterns entries including CONTRIBUTING.rst and doc/source/development/core_developer.md.
Label instruction updates
\.coderabbit.yaml
Appended "Mutually exclusive." to descriptions for: API, New feature, Maintenance, Bug fix, Documentation, Infrastructure, Enhancement, Performance.
Manifest summary
\.coderabbit.yaml
Net diff: +23 / -8 lines (configuration metadata changes).

Sequence Diagram(s)

(omitted — configuration-only changes, no multi-component control flow to visualize)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • lagru
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise (60 characters, under 80), descriptive, and clearly summarizes the main change of adjusting CodeRabbit configuration settings to fit the development style.
Description check ✅ Passed The description is related to the changeset, containing three bullet points that correspond to the actual changes made: mutually exclusive labels, restricted bot responses, and contributor guidelines integration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In @.coderabbit.yaml:
- Around line 27-29: Update the Enhancement label entry so it explicitly states
that the label is mutually exclusive with other type labels: locate the label
line that currently reads ":fast_forward: type: Enhancement" and append the same
mutual-exclusivity note used by other type labels to its instructions text so
consistency with the PR objective is maintained.
- Around line 43-47: Update the YAML key under knowledge_base.code_guidelines
from file_patterns to the schema-compliant camelCase name filePatterns so
CodeRabbit will ingest the listed files; specifically change the key named
file_patterns to filePatterns while keeping the same values (e.g.,
CONTRIBUTING.rst and doc/source/development/core_developer.rst) intact.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Copy link
Copy Markdown
Member

@lagru lagru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's try it. Thanks @stefanv.

@lagru lagru merged commit dce8cc5 into scikit-image:main Jan 27, 2026
26 checks passed
@stefanv stefanv added this to the 0.27 milestone Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖 type: Infrastructure CI, packaging, tools and automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants