Skip to content

chore: add Robie and Loïc to CODEOWNERS - #166

Open
Christopher Obbard (obbardc) wants to merge 1 commit into
mainfrom
wip/obbardc/codeowners-more-reviewers
Open

Christopher Obbard (obbardc) wants to merge 1 commit into
mainfrom
wip/obbardc/codeowners-more-reviewers

Conversation

@obbardc

Copy link
Copy Markdown
Contributor

Only two people are asked to review pull requests into main, so a change can sit waiting whenever either of them is busy or away.

Add Robie Basak (@basak-qcom) and Loïc Minier (@lool) to the rule so GitHub requests them too, widening the pool of reviewers. Both have an admin role on the repository which GitHub requires for an owner to take effect. CODEOWNERS has no notion of primary and secondary owners: everyone listed is requested and an approval from any one of them is enough, so they are appended after the existing owners.

I have left this as a draft as it will add you both to ALL future PRs. Perhaps we don't want to do this, but I am opening this PR to start the discussion :-).

@basak-qcom

Copy link
Copy Markdown

I'm happy to be a backup. I expect I'll generally defer to you and Bjordis when you're available though. I'm really happy with the progress you've both made in this codebase and would be worried about stepping on your toes.

Only two people are asked to review pull requests into main, so a
change can sit waiting whenever either of them is busy or away.

Add Robie Basak (basak-qcom) and Loïc Minier (lool) to the rule so
GitHub requests them too, widening the pool of reviewers. Both have an
admin role on the repository which GitHub requires for an owner to
take effect. CODEOWNERS has no notion of primary and secondary owners:
everyone listed is requested and an approval from any one of them is
enough, so they are appended after the existing owners.

Signed-off-by: Christopher Obbard <chris.obbard@oss.qualcomm.com>

This branch was successfully deployed

1 active deployment
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.

2 participants