CRSU-Apps/MetaInsight

Add Branch Protection

RyanJField opened this issue · 4 comments

Add branch protections rules to the main branch, require a pull request and at least 1 reviewer.

@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.

Agreed. I think that 2 reviewers are only needed for big changes or where nobody has sufficient expertise in the area on their own.

Closing as agreement on 1 reviewer.