Require Github PR approvals from multiple groups

Question

I'm using the codeowners file to require PR approvals before they can be merged. What I'd like to do is:

Require all PRs to have at least one approval from a group of code owners (engineers)
Require all PRs to have at least one approval from a group of QA

It seems like the code owners file does a hierarchical thing where only one group owns the code and you can create rules for certain directories but those will just override the default code owner.

My current .github/CODEOWNERS file looks like this:

* @my-org/engineer-code-owners

Is there a way to require at least one approval from two different groups?

superhawk610 · Answer 1 · 2021-03-27T02:48:31.783

7

You can list multiple users/groups on a single line (separated by a single space) to request multiple reviews.

* @my-org/engineer-code-owners @my-org/qa-code-owners

As of time of writing, there’s no way to require a review from all code owners assigned to a pull request.

Is there any way to require all of the listed people to approve?

No, there currently isn’t a way to do that built-in to the CODEOWNERS feature.

See CODEOWNERS reference and thread on required reviews.

edited Mar 27 '21 at 02:48

answered Mar 26 '21 at 18:17

superhawk610

2,457
2
18
27

2

And would this allow me to require at least one review from _each_ group? – Brady Dowling Mar 27 '21 at 00:21
2

@brady-dowling unfortunately no. Updated my answer to reflect this. – superhawk610 Mar 27 '21 at 02:49
3

grrrrr - I wanted that too!!! – Dean Hiller Sep 15 '22 at 14:09

score 4 · Answer 2 · answered Feb 23 '23 at 14:13

On one line of CODEOWNERS, you can request reviews from multiple reviewers but you can't require approvals.

In the repo settings, branch protection, you can require a certain number of approvals but you can't specify who.

I was really hoping I could stop myself merging without QA approval!

Require Github PR approvals from multiple groups

2 Answers2