Closed
Description
Hi! Happy to see activity in this repo!
So, I'd propose we enable some branch protection rules for the master branch:
- Require PR reviews before merging
- Require status checks to pass before merging
- (open to debate) Require liner history
- Enforce these rules on admins as well
On top of this, we might want to disable merge commits and rebase merging, to ensure the PR author:
- Takes care of rebasing against the latest master
- We end up with 1 commit for each PR, so it's easier to track what was done in that PR (easier cherry-pick, reverts...)
What do you think?
@olleolleolle @skywinder (and others! I lost track of the names of new maintainers, sorry! feel free to chime in :)