Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add CODEOWNERS #1001

Merged
merged 1 commit into from
Nov 14, 2024
Merged

add CODEOWNERS #1001

merged 1 commit into from
Nov 14, 2024

Conversation

jmartin-tech
Copy link
Collaborator

Enforce approval requirements for changes that may impact automation.

enforce CODEOWNERS on .github files to restirct workflow and template
changes as requiring approval by maintainers

Signed-off-by: Jeffrey Martin <[email protected]>
@jmartin-tech
Copy link
Collaborator Author

See docs

@@ -0,0 +1 @@
/.github/ @leondz @erickgalinkin @jmartin-tech
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we also want to add ourselves as global owners for review tagging? Or are we only aiming for branch protection for the CODEOWNERS file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I would say no, I do not currently want all PRs to auto-request review.

This is intended for protection of automation artifacts by requiring a maintainer to approve landing any PR that may change those workflows. I can see a future where there are experts on certain segments of the codebase that would be desirable to tag on reviews of changes in those areas, the project is not that complex at this time.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think having a .github restriction, separate from global config/restrictions, is a good extra layer of protection

Copy link
Collaborator

@leondz leondz left a comment

Choose a reason for hiding this comment

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

nice

@@ -0,0 +1 @@
/.github/ @leondz @erickgalinkin @jmartin-tech
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think having a .github restriction, separate from global config/restrictions, is a good extra layer of protection

@jmartin-tech jmartin-tech merged commit c775303 into NVIDIA:main Nov 14, 2024
7 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 14, 2024
@jmartin-tech jmartin-tech deleted the task/codeowners branch November 14, 2024 17:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants