Add Branch Protection
RyanJField opened this issue · 4 comments
RyanJField commented
Add branch protections rules to the main branch, require a pull request and at least 1 reviewer.
RyanJField commented
@CRSU-Apps/devs I have added branch protections rules, does anyone have any comments about the number of minimum reviewers, currently this is set to 1. This can be increased or we could have an informal rule stating don't merge unless all requested reviewers have approved the PR.
nabury commented
I think 1 reviewer as the minimum is fine - for small updates that's often all you need.
JanionNevill commented
Agreed. I think that 2 reviewers are only needed for big changes or where nobody has sufficient expertise in the area on their own.
RyanJField commented
Closing as agreement on 1 reviewer.