What should the branch protection rules for main be?

[nikosbosse]

Currently merging on main is blocked unless we have one approving review. That approval stands at the moment even if someone pushes a new commit to the PR.

[nikosbosse]

@seabbs @sbfnk

[sbfnk]

I'd say yes as long as you can override when reviewers are unresponsive.

[seabbs]

I'd say yes as well but keep admins being able to override as needed (so you can push through things you want).

[nikosbosse]

I'm the entire reason we need branch protection 🥲

[seabbs]

having to tick yes skip them is meant to make you really think if its a good idea...