Skip to content

ci: add CodeRabbit review configuration - #348

Merged
rapids-bot[bot] merged 4 commits into
rapidsai:mainfrom
johanpel:coderabbit
Jul 10, 2026
Merged

ci: add CodeRabbit review configuration#348
rapids-bot[bot] merged 4 commits into
rapidsai:mainfrom
johanpel:coderabbit

Conversation

@johanpel

@johanpel johanpel commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

Add a repo-root .coderabbit.yaml customizing CodeRabbit for quent's layout (Rust crates, TS/React ui, proto, docs, workflows). markdownlint is disabled since Markdown is already linted in CI via rumdl.

Add a repo-root .coderabbit.yaml customizing CodeRabbit for quent's layout
(Rust crates, TS/React ui, proto, docs, workflows). markdownlint is disabled
since Markdown is already linted in CI via rumdl.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@johanpel

johanpel commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

coderabbitai[bot]

This comment was marked as outdated.

johanpel and others added 2 commits July 9, 2026 20:21
Scope the proto and docs path_instructions to their intended file types
(proto/**/*.proto, docs/**/*.md) so they no longer match Cargo.toml, lib.rs,
book.toml, or .gitignore. Point the schema reference at the official
coderabbit.ai endpoint.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@johanpel
johanpel marked this pull request as ready for review July 10, 2026 07:12
@rapidsai rapidsai deleted a comment from coderabbitai Bot Jul 10, 2026
@rapidsai rapidsai deleted a comment from coderabbitai Bot Jul 10, 2026
@johanpel

johanpel commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

Updates .coderabbit.yaml with quiet review behavior, excluded paths, detailed Rust/UI/docs/workflow guidance, and restricted knowledge-base inputs.

Changes

CodeRabbit Review Configuration

Layer / File(s) Summary
Review behavior and scope
.coderabbit.yaml
Configures the quiet profile, walkthrough and review-status behavior, branch and bot filters, and exclusions for generated TypeScript and lockfiles.
Path guidance and knowledge base
.coderabbit.yaml
Adds review rules for Rust, UI, documentation, and GitHub workflows, while restricting code-guideline inputs to CONTRIBUTING.md.

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

A rabbit checks the YAML lines,
With carrot marks and tidy signs.
Rust hops left, UI hops right,
Docs and workflows shine bright.
Quiet reviews now nap tonight.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes only the Description section and omits the required Related Issues, Testing, and Screenshots sections. Add the missing template sections and include testing commands, related issue links, and screenshots or note why they are not needed.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title is concise and accurately describes the main change: adding a CodeRabbit review configuration.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@johanpel

Copy link
Copy Markdown
Contributor Author

Seems like coderabbit won't work with its own modified config, probably for security reasons. In this PR it's probably just showing its default behavior which is rather noisy and should be surpressed by the proposed config. So we'll notice whether this minimal config is not too noisy after merge.

@johanpel

Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot
rapids-bot Bot merged commit f97976a into rapidsai:main Jul 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants