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

RFC: Initial pass at a CODEOWNERS #458

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

RFC: Initial pass at a CODEOWNERS #458

wants to merge 1 commit into from

Conversation

JoshStrobl
Copy link
Member

@JoshStrobl JoshStrobl commented Sep 23, 2023

This pull request is an RFC to adds codeowner defaults for various parts of the codebase, based on historical or regular contributions. It is NOT intended to be used to enforce or mandate blocking individuals for merge. Listings can / SHOULD be modified based on interest / desire for code reviews, and SHOULD NOT be considered exhaustive.

All listings have an addition of the Best Buds team, so anyone on the team WILL be notified automatically. Global rule notifies the Best Buds team as well.

@JoshStrobl
Copy link
Member Author

Modification by reviews can be done directly to branch after discussion here or in Matrix, avoiding additional noise from shifting things, adding or removing entries, etc.

@JoshStrobl
Copy link
Member Author

Noticed missing language / words in original commit message. Updated to reflect it is not intend to enforce / mandate blockers, and uses common SHOULD / SHOULD NOT / NOT / WILL language you typically see in other RFCs or specs.

This pull request is an RFC to adds codeowner defaults for various parts of the codebase, based on historical or regular contributions. It is intended to be used to enforce or mandate blocking individuals for merge. Listings can / should be modified based on interest / desire for code reviews, not necessarily considered exhaustive.

All listings have an addition of the Best Buds team, so anyone on the team should be notified automatically. Global rule notifies the Best Buds team as well.
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! :)

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