ci: Add CodeRabbit configuration file by chtruong814 · Pull Request #1063 · NVIDIA-NeMo/Skills

chtruong814 · 2025-12-02T17:21:41Z

test

Summary by CodeRabbit

Chores
- Added configuration file for code review automation and knowledge base settings, including review profile preferences, pre-merge checks, and code guideline references.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Signed-off-by: Charlie Truong <chtruong@nvidia.com>

chtruong814 · 2025-12-02T17:31:11Z

@coderabbitai review

coderabbitai · 2025-12-02T17:31:23Z

✅ Actions performed

Review triggered.

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 · 2025-12-02T17:33:37Z

📝 Walkthrough

Walkthrough

A new .coderabbit.yaml configuration file has been added to specify CodeRabbit review settings, including language preferences, review profiles, auto-review configurations, knowledge base settings with code guidelines, and file patterns for the repository.

Changes

Cohort / File(s)	Summary
Configuration `.coderabbit.yaml`	Adds CodeRabbit review configuration with language set to "en-US", review profile "chill", auto-title generation, pre-merge checks, auto-review settings, and knowledge base configuration referencing "CONTRIBUTING.md" as the guidelines file.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Verify that all configuration values align with intended CodeRabbit behavior and project policies
Confirm that base_branches override and knowledge base filePatterns are appropriate for the repository structure
Validate the auto_review and pre-merge check settings match project requirements

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The pull request title '@coderabbitai title' is a vague command rather than a descriptive summary of the actual changes, which involve adding a .coderabbit.yaml configuration file with review settings.	Replace the title with a descriptive summary of the change, such as 'Add .coderabbit.yaml configuration with review settings' or similar that clearly indicates the pull request adds a configuration file.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch chtruong/debug-rabbit

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 and usage tips.}

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5e8e26a and 58f8d99.

📒 Files selected for processing (1)

nemo_skills/file_utils.py (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

GitHub Check: unit-tests

nemo_skills/file_utils.py

chtruong814 · 2025-12-02T19:50:11Z

@coderabbitai resume

coderabbitai · 2025-12-02T19:50:18Z

✅ Actions performed

Reviews resumed.

Signed-off-by: Charlie Truong <chtruong@nvidia.com>

Signed-off-by: George Armstrong <georgea@nvidia.com>

.coderabbit.yaml

Signed-off-by: George Armstrong <georgea@nvidia.com>

gwarmstrong · 2025-12-02T21:04:38Z

@coderabbitai review

coderabbitai · 2025-12-02T21:04:56Z

✅ Actions performed

Review triggered.

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

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 58f8d99 and 439f510.

📒 Files selected for processing (1)

.coderabbit.yaml (1 hunks)

🧰 Additional context used

🪛 GitHub Actions: Lint and Format

.coderabbit.yaml

[error] 1-1: Pre-commit end-of-file-fixer failed. Files were modified by this hook (e.g., .coderabbit.yaml). No newline at end of file.

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

GitHub Check: unit-tests

🔇 Additional comments (2)

.coderabbit.yaml (2)

68-68: Prior review suggestions have been properly addressed.

The configuration file successfully incorporates all of gwarmstrong's suggestions:

Line 68: custom_checks: [] implemented

Line 80: base_branches: ["main", "chtruong/*"] implemented

Lines 90–93: filePatterns: ["CONTRIBUTING.md"] implemented

Also applies to: 80-80, 90-93

59-59: Verify intentional changes to review feature toggles.

Several features have been disabled from their defaults: poem (false), review_status (false), and auto_incremental_review (false). The commit_status is also disabled. Please confirm these are intentional configuration choices for your workflow, as disabling review_status may reduce visibility into review progress and disabling auto_incremental_review may require manual review triggers on each push.

Also applies to: 62-62, 74-74

coderabbitai · 2025-12-02T21:08:55Z

.coderabbit.yaml

+    filePatterns: # current: []
+      # File Patterns - Specify files for your coding guideline documents in this section. CodeRabbit will scan these files to understand your team's standards and apply them during code reviews. Multiple files supported. File names are case-sensitive. Common files like: (**/.cursorrules, .github/copilot-instructions.md, .github/instructions/*.instructions.md, **/CLAUDE.md, **/GEMINI.md, **/.cursor/rules/*, **/.windsurfrules, **/.clinerules/*, **/.rules/*, **/AGENT.md, **/AGENTS.md) are included by default.
+      # Default: []
+      - "CONTRIBUTING.md"


⚠️ Potential issue | 🔴 Critical

Add newline at end of file to fix pre-commit hook failure.

The pre-commit end-of-file-fixer failed because the file is missing a trailing newline. This is blocking the pipeline.

Add a newline at the end of line 93:

- "CONTRIBUTING.md" +

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

- "CONTRIBUTING.md"

- "CONTRIBUTING.md"

🤖 Prompt for AI Agents

.coderabbit.yaml around line 93: the file is missing a trailing newline which causes the pre-commit end-of-file-fixer to fail; open .coderabbit.yaml and add a single newline character after line 93 (after the "CONTRIBUTING.md" entry) so the file ends with a newline, then save and re-run pre-commit.

.coderabbit.yaml

Signed-off-by: George Armstrong <georgea@nvidia.com>

Signed-off-by: Charlie Truong <chtruong@nvidia.com> Signed-off-by: George Armstrong <georgea@nvidia.com> Co-authored-by: George Armstrong <georgea@nvidia.com> Signed-off-by: George Armstrong <georgea@nvidia.com>

Signed-off-by: Charlie Truong <chtruong@nvidia.com> Signed-off-by: George Armstrong <georgea@nvidia.com> Co-authored-by: George Armstrong <georgea@nvidia.com> Signed-off-by: mmkrtchyan <mmkrtchyan@nvidia.com>

Signed-off-by: Charlie Truong <chtruong@nvidia.com> Signed-off-by: George Armstrong <georgea@nvidia.com> Co-authored-by: George Armstrong <georgea@nvidia.com>

Signed-off-by: Charlie Truong <chtruong@nvidia.com> Signed-off-by: George Armstrong <georgea@nvidia.com> Co-authored-by: George Armstrong <georgea@nvidia.com> Signed-off-by: dgitman <dgitman@nvidia.com>

Debug

58f8d99

Signed-off-by: Charlie Truong <chtruong@nvidia.com>

coderabbitai bot reviewed Dec 2, 2025

View reviewed changes

nemo_skills/file_utils.py Outdated Show resolved Hide resolved

chtruong814 and others added 2 commits December 2, 2025 14:01

Add code rabbit config

969c84f

Signed-off-by: Charlie Truong <chtruong@nvidia.com>

Apply suggestion from @gwarmstrong

500837d

Signed-off-by: George Armstrong <georgea@nvidia.com>

gwarmstrong reviewed Dec 2, 2025

View reviewed changes

.coderabbit.yaml Outdated Show resolved Hide resolved

gwarmstrong reviewed Dec 2, 2025

View reviewed changes

.coderabbit.yaml Outdated Show resolved Hide resolved

gwarmstrong reviewed Dec 2, 2025

View reviewed changes

.coderabbit.yaml Outdated Show resolved Hide resolved

Apply suggestions from code review

439f510

Signed-off-by: George Armstrong <georgea@nvidia.com>

gwarmstrong changed the title ~~Debug PR review~~ @coderabbitai title Dec 2, 2025

coderabbitai bot changed the title ~~@coderabbitai title~~ ci: Add CodeRabbit configuration file Dec 2, 2025

coderabbitai bot reviewed Dec 2, 2025

View reviewed changes

gwarmstrong reviewed Dec 2, 2025

View reviewed changes

.coderabbit.yaml Outdated Show resolved Hide resolved

Apply suggestion from @gwarmstrong

bc3903d

Signed-off-by: George Armstrong <georgea@nvidia.com>

gwarmstrong approved these changes Dec 2, 2025

View reviewed changes

gwarmstrong enabled auto-merge (squash) December 2, 2025 21:11

gwarmstrong merged commit 29674a4 into main Dec 2, 2025
5 checks passed

gwarmstrong deleted the chtruong/debug-rabbit branch December 2, 2025 21:26

wasiahmad pushed a commit that referenced this pull request Feb 4, 2026

ci: Add CodeRabbit configuration file (#1063)

aff6e4a

Signed-off-by: Charlie Truong <chtruong@nvidia.com> Signed-off-by: George Armstrong <georgea@nvidia.com> Co-authored-by: George Armstrong <georgea@nvidia.com>

dgtm777 pushed a commit that referenced this pull request Mar 18, 2026

ci: Add CodeRabbit configuration file (#1063)

3b723d9

Signed-off-by: Charlie Truong <chtruong@nvidia.com> Signed-off-by: George Armstrong <georgea@nvidia.com> Co-authored-by: George Armstrong <georgea@nvidia.com>

Conversation

chtruong814 commented Dec 2, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

chtruong814 commented Dec 2, 2025

Uh oh!

coderabbitai bot commented Dec 2, 2025

Uh oh!

coderabbitai bot commented Dec 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

chtruong814 commented Dec 2, 2025

Uh oh!

coderabbitai bot commented Dec 2, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gwarmstrong commented Dec 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai bot commented Dec 2, 2025

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Dec 2, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

chtruong814 commented Dec 2, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Dec 2, 2025 •

edited

Loading

gwarmstrong commented Dec 2, 2025 •

edited

Loading