mergeable icon indicating copy to clipboard operation
mergeable copied to clipboard

Published 20 hours ago • verified publisher icon mergeability

Feature: Require all codeowners

Open Dylan-Prins opened this issue 1 year ago • 4 comments

I would love to have the option to require a review from all the codeowners. That way I can have different teams review the PR. Like Engineering and Risk

Dylan-Prins avatar Mar 02 '23 11:03 Dylan-Prins

Are you using the team or teams check?

dns2utf8-tipi avatar Aug 31 '23 15:08 dns2utf8-tipi

Team

Dylan-Prins avatar Sep 06 '23 06:09 Dylan-Prins

I used an or stack to check for either group/team. I think you have to require a number of reviewer that you have to adjust according to the total number of members in all teams

dns2utf8-tipi avatar Sep 06 '23 12:09 dns2utf8-tipi

Checkout this implementation: https://github.com/tipi-oss-testing/tipi-oss-test/blob/main/.github/mergeable.yml

dns2utf8-tipi avatar Sep 08 '23 13:09 dns2utf8-tipi