Skip to content

Overhaul ops-codeowners coverage - #15660

Merged
raydouglass merged 1 commit into
NVIDIA:branch-24.06from
raydouglass:ops-codeowners
May 9, 2024
Merged

Overhaul ops-codeowners coverage#15660
raydouglass merged 1 commit into
NVIDIA:branch-24.06from
raydouglass:ops-codeowners

Conversation

@raydouglass

Copy link
Copy Markdown
Contributor

This PR overhauls how ops-codeowners reviews are handled.

ops-codeowners is replaced by ci-codeowners & packaging-codeowners. The coverage of files is expanded as well.

Additionally, the process will change: reviews will be assigned to a member of the teams instead of a manual request to ops-codeowners.

@raydouglass raydouglass added feature request New feature or request non-breaking Non-breaking change labels May 6, 2024

@bdice bdice left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks great.

@ajschmidt8 ajschmidt8 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.

LGTM

@raydouglass
raydouglass marked this pull request as ready for review May 9, 2024 19:06
@raydouglass
raydouglass requested a review from a team as a code owner May 9, 2024 19:06
@raydouglass
raydouglass merged commit bd93e20 into NVIDIA:branch-24.06 May 9, 2024
@raydouglass
raydouglass deleted the ops-codeowners branch May 9, 2024 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature request New feature or request non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants