Add Branch Protection

Question

Add Branch Protection

Closed this issue a year ago · 4 comments

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

Answer 1 · 2023-06-30T13:25:47.000Z

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

Answer 2 · 2023-06-30T13:32:13.000Z

I think 1 reviewer as the minimum is fine - for small updates that's often all you need.

Answer 3 · 2023-06-30T13:58:53.000Z

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

Answer 4 · 2023-06-30T14:20:10.000Z

Closing as agreement on 1 reviewer.