Skip to content

chore: add CODEOWNERS for automated PR review routing#894

Merged
cv merged 3 commits into
mainfrom
chore/add-codeowners-v2
Mar 25, 2026
Merged

chore: add CODEOWNERS for automated PR review routing#894
cv merged 3 commits into
mainfrom
chore/add-codeowners-v2

Conversation

@cv
Copy link
Copy Markdown
Collaborator

@cv cv commented Mar 25, 2026

Summary

  • Adds .github/CODEOWNERS to auto-assign reviewers by file path
  • Routes CLI/scripts/CI/infra changes to @NVIDIA/nemoclaw-maintainer
  • Routes docs/tests/onboarding to @NVIDIA/nemoclaw-engineer
  • Routes policy files and Dockerfile to @NVIDIA/nemoclaw-security
  • GitHub round-robins within each team to spread review load

Context

The open PR queue has grown steadily, with 88% stuck at REVIEW_REQUIRED. This file ensures every PR gets routed to the right team automatically.

Note

For full enforcement, the org-level ruleset on main should have "Require review from Code Owners" enabled. Without that flag, any team member can approve regardless of path ownership.

Test plan

  • Verify CODEOWNERS syntax is valid (GitHub will show a warning banner on the file if not)
  • Open a test PR touching docs/ and confirm nemoclaw-engineer is auto-assigned
  • Open a test PR touching nemoclaw/src/ and confirm nemoclaw-maintainer is auto-assigned

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated repository governance configuration for pull request reviewer assignment.

cv and others added 3 commits March 25, 2026 07:13
Move post_install_message() call after onboarding so PATH hints stay
visible to the user.

Co-authored-by: Hagege Ruben <20857346+HagegeR@users.noreply.github.com>
Routes PRs to the right team based on file paths:
- nemoclaw-maintainer: CLI, scripts, CI, infra
- nemoclaw-engineer: docs, tests, onboarding
- nemoclaw-security: policies, Dockerfile

GitHub will round-robin within each team to spread review load.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 69969c97-67ce-4ec8-8058-0e19f72b7755

📥 Commits

Reviewing files that changed from the base of the PR and between 5748a32 and 8e9f201.

📒 Files selected for processing (1)
  • .github/CODEOWNERS

📝 Walkthrough

Walkthrough

A .github/CODEOWNERS file is added to define code ownership rules across the repository. It establishes default ownership and path-specific assignments for CLI plugins, blueprints, policies, scripts, containers, documentation, tests, and configuration directories.

Changes

Cohort / File(s) Summary
GitHub Configuration
.github/CODEOWNERS
New file establishing code ownership rules with fallback team assignment and path-specific ownership mappings for CLI, blueprints, security policies, scripts, containers, documentation, tests, and repository configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 The warren grows and ownership is clear,
With paths and rules for those held dear,
Each team now knows what's in their care,
No confusion in the maintainer's lair! 🏘️

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: add CODEOWNERS for automated PR review routing' clearly and concisely summarizes the main change—adding a CODEOWNERS file for automated PR review assignment.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/add-codeowners-v2

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

@cv cv merged commit ca92680 into main Mar 25, 2026
10 checks passed
@cv cv deleted the chore/add-codeowners-v2 branch March 25, 2026 16:52
lakamsani pushed a commit to lakamsani/NemoClaw that referenced this pull request Apr 4, 2026
* refactor(install): show post-install message last for visibility (NVIDIA#847)

Move post_install_message() call after onboarding so PATH hints stay
visible to the user.

Co-authored-by: Hagege Ruben <20857346+HagegeR@users.noreply.github.com>

* chore: add CODEOWNERS for automated PR review routing

Routes PRs to the right team based on file paths:
- nemoclaw-maintainer: CLI, scripts, CI, infra
- nemoclaw-engineer: docs, tests, onboarding
- nemoclaw-security: policies, Dockerfile

GitHub will round-robin within each team to spread review load.

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

---------

Co-authored-by: Hagege Ruben <20857346+HagegeR@users.noreply.github.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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