-
Notifications
You must be signed in to change notification settings - Fork 1
Add CodeRabbit config to include all paths for review #20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
📝 WalkthroughWalkthroughA new configuration file Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
@coderabbitai config |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain@russellhaering I'll fetch the current CodeRabbit configuration and schema information for this repository to help address the issue with changes being ignored. 🛠 Debug Information Gathered @russellhaering I can see there's a configuration error in your 🏁 Script executed: Length of output: 49 @russellhaering Let me check what files are being added in this PR: 🏁 Script executed: Length of output: 289 @russellhaering I found the issue! The The ProblemYour current configuration: reviews:
path_filters:
included_paths:
- "**/*"
excluded_paths: []This is causing the validation error: The SolutionTo review all files in the repository, simply use an empty array: reviews:
path_filters: []Alternatively, if you want to explicitly include all files, you can use: reviews:
path_filters:
- "**/*"How path_filters Works
Examples: # Review only specific directories
path_filters:
- "src/**"
- "lib/**"
# Review everything except certain patterns
path_filters:
- "**/*"
- "!dist/**"
- "!node_modules/**"
# Review everything (simplest)
path_filters: []Since your goal is to "make CodeRabbit not ignore every change in this repo," I recommend using the empty array approach: |
This docs repo was being ignored by CodeRabbit due to path filters. Configure CodeRabbit to review all files including .mdx documentation.
3957333 to
fa95917
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
.coderabbit.yaml (1)
1-3: Consider using an empty array for clearer intent.While the current configuration with
"**/*"is valid and will include all files, using an empty array is the recommended approach for reviewing all files. This is simpler, more explicit about the intent, and aligns with CodeRabbit's best practices.♻️ Recommended simplification
reviews: - path_filters: - - "**/*" + path_filters: []This achieves the same result (reviewing all files) with a clearer, more concise syntax. The empty array explicitly signals "no filters, review everything."
Try to make CodeRabbit not ignore every change in this repo.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.