Skip to content

chore: [CODEOWNERS] use gh teams instead of usernames - #2941

Open
tariq1890 wants to merge 1 commit into
mainfrom
add-team-to-codeowners
Open

tariq1890 wants to merge 1 commit into
mainfrom
add-team-to-codeowners

Conversation

@tariq1890

@tariq1890 tariq1890 commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Instead of listing every GitHub username as an authorised code owner, we use a GitHub team instead. This has two advantages:

  • No need to update CODEOWNERS when members are added or removed
  • It keeps the added reviewer list for PRs minimal

Instead of listing every GitHub username as an authorised code owner,
we use a GitHub team instead. This has two advantages:

- No need to update CODEOWNERS when members are added or removed
- It keeps the added reviewer list to PRs small

Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: NVIDIA/gpu-operator/.coderabbit.yaml

Review profile: QUIET

Plan: Enterprise

Run ID: 75957d7e-1c8d-4f60-91f2-b13cb69023b4

📥 Commits

Reviewing files that changed from the base of the PR and between bf73d07 and 840faa2.

📒 Files selected for processing (1)
  • CODEOWNERS

Included review availability: Your plan provides up to 12 included reviews per hour; 11 remain after this review.


📝 Walkthrough

Walkthrough

The CODEOWNERS entry now assigns general approval for the gpu-operator codebase to @NVIDIA/gpu-operator-dev instead of six individual maintainers.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 840fa

This change updates CODEOWNERS to use the gpu-operator-dev team. No concrete merge-blocking risk is established by the supplied evidence.


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

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 44.49%. remained the same — add-team-to-codeowners into main

@rahulait

rahulait commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Does it auto-detect who the users are in that mailing list?
nvm, I see the team now

This branch has not been deployed

No deployments
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.

3 participants