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 .github/CODEOWNERS #488

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

themr0c
Copy link
Member

@themr0c themr0c commented Sep 12, 2024

Adding a CODEOWNSERS file would help GitHub propose more accurate reviewers when you open a pull request.

Currently a draft, to check if it is something we can implement.
To make it work, engineers require write permissions on the repository.

Signed-off-by: Fabrice Flore-Thébault <[email protected]>
@rhdh-bot
Copy link
Collaborator

.github/CODEOWNERS Outdated Show resolved Hide resolved
.github/CODEOWNERS Outdated Show resolved Hide resolved
@rhdh-bot
Copy link
Collaborator

@nickboldt
Copy link
Member

nickboldt commented Sep 24, 2024

On hold for a couple reasons:

  • team reorg is happening Real Soon Now (next sprint?)
  • creation/auditing of existing rhd/rhdh groups needs to happen to clean things up, make them consistent, and document maintenance steps

Once the above is sorted we can switch this PR to use groups of SMEs instead of individual SMEs.

See also https://redhat-internal.slack.com/archives/C04CUSD4JSG/p1726226954447179?thread_ts=1726065087.941969&cid=C04CUSD4JSG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants