Skip to content

Update CODEOWNERS team names for rapidsai->NVIDIA migration#2261

Merged
rapids-bot[bot] merged 3 commits into
NVIDIA:mainfrom
josephine-wolf-oberholtzer:josephine/migrate-codeowners
Jun 24, 2026
Merged

Update CODEOWNERS team names for rapidsai->NVIDIA migration#2261
rapids-bot[bot] merged 3 commits into
NVIDIA:mainfrom
josephine-wolf-oberholtzer:josephine/migrate-codeowners

Conversation

@josephine-wolf-oberholtzer

Copy link
Copy Markdown
Collaborator

XREF: https://github.com/rapidsai/build-infra/issues/367

After we migrate cuVS to the NVIDIA GH org, we'll need to update the CODEOWNERS team names to ensure continuity.

N.B. DO NOT MERGE until after this repo has been migrated to the NVIDIA org.

@josephine-wolf-oberholtzer

Copy link
Copy Markdown
Collaborator Author

FWIW the verify-codeowners pre-commit check is desperately trying to revert this. She won't pass until probably post-org-migration.

@jameslamb

Copy link
Copy Markdown
Member

FWIW the verify-codeowners pre-commit check is desperately trying to revert this. She won't pass until probably post-org-migration.

Just delete it from .pre-commit-config.yaml for now. It's there to enforce conventions and we're changing the conventions intentionally here.

@jameslamb jameslamb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will CODEOWNERS assignment work cross-org? Like are we expecting that if this is merged, auto-assignment will continue to work for as long as the repo's still here in the rapidsai/ org?

I genuinely don't know, and if that works then that's pretty cool!

If it doesn't, would wait to do this until after the repo moves.

@josephine-wolf-oberholtzer

Copy link
Copy Markdown
Collaborator Author

@jameslamb AFAIK no team-based anything works across orgs. Per PR description I don't want this merged until after the migration happens.

That said, I kicked off the migration paperwork a couple minutes ago, so I think this PR is appropriately tee'd up. I'll re-run the checks post-migration.

@jameslamb

Copy link
Copy Markdown
Member

🙈 sorry for misreading the description, ok great let's wait til it moves.

@cjnolet cjnolet added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Jun 24, 2026
@josephine-wolf-oberholtzer

Copy link
Copy Markdown
Collaborator Author

/ok to test a0d9d07

1 similar comment
@josephine-wolf-oberholtzer

Copy link
Copy Markdown
Collaborator Author

/ok to test a0d9d07

@josephine-wolf-oberholtzer

Copy link
Copy Markdown
Collaborator Author

/merge

1 similar comment
@josephine-wolf-oberholtzer

Copy link
Copy Markdown
Collaborator Author

/merge

@rapids-bot rapids-bot Bot merged commit c95fc4e into NVIDIA:main Jun 24, 2026
55 checks passed
@josephine-wolf-oberholtzer josephine-wolf-oberholtzer deleted the josephine/migrate-codeowners branch June 24, 2026 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants