Skip to content

chore(codeowners): assign infra tree - #667

Merged
kristinapathak merged 1 commit into
mainfrom
kpathak/codeowners-infra
Aug 4, 2026
Merged

chore(codeowners): assign infra tree#667
kristinapathak merged 1 commit into
mainfrom
kpathak/codeowners-infra

Conversation

@kristinapathak

@kristinapathak kristinapathak commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

TL;DR

Route every path below /infra/ to the Infrastructure team.

Additional Details

The prior rule matched only /infra/cassandra/, leaving new infrastructure paths to the default owners. This expands the existing routing rule without changing ownership of current paths.

For the Reviewer

Review .github/CODEOWNERS.

For QA

Ran git diff --check.

QA is not needed for this ownership-only metadata change.

Issues

Closes #666

Checklist

  • I am familiar with the Contributing Guidelines.
  • I have signed off my commits for Developer Certificate of Origin (DCO) compliance.
  • New or existing tests cover these changes. Not applicable: ownership-only metadata change.
  • The documentation is up to date with these changes.

Signed-off-by: Kristina Pathak <kpathak@nvidia.com>
@kristinapathak
kristinapathak requested a review from a team as a code owner August 4, 2026 19:44
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The CODEOWNERS rule changed from /infra/cassandra/ to /infra/. This assigns infrastructure-team ownership to all current and future paths under /infra/.

Changes

Infrastructure ownership

Layer / File(s) Summary
Expand infrastructure ownership scope
.github/CODEOWNERS
The infrastructure ownership pattern now covers the full /infra/ directory instead of only /infra/cassandra/.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: famousdirector

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The change replaces /infra/cassandra/ with /infra/ and assigns all infrastructure paths to @NVIDIA/nvcf-infra-dev as required by issue #666.
Out of Scope Changes check ✅ Passed The pull request changes only the CODEOWNERS rule required to expand infrastructure ownership coverage.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title uses valid Conventional Commits syntax and accurately describes the CODEOWNERS ownership update.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch kpathak/codeowners-infra

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

@kristinapathak
kristinapathak added this pull request to the merge queue Aug 4, 2026
Merged via the queue into main with commit bfcc148 Aug 4, 2026
16 checks passed
@kristinapathak
kristinapathak deleted the kpathak/codeowners-infra branch August 4, 2026 20:01
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.

chore(codeowners): assign all infra paths to Infra

2 participants