-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Problem
Branch protection is enabled on main, but the current policy is minimal and does not enforce meaningful review gates:
- Required approving reviews: 0 — PRs can be merged without any approval
- Dismiss stale reviews: No
- Require code owner reviews: No
- Require signed commits: No
- Enforce for admins: No
This effectively allows direct merges without review.
Recommended Changes
Update the branch protection rule on main to include:
- Require at least 1 approving review
- Dismiss stale pull request approvals when new commits are pushed
- Do not allow force pushes (already set)
- Do not allow deletions (already set)
- Consider: Require code owner reviews
- Consider: Require signed commits
- Consider: Include administrators in these restrictions
How To Update
- Go to Settings > Branches
- Edit the existing
mainprotection rule - Under "Require a pull request before merging", set Required number of approvals to at least 1
- Enable Dismiss stale pull request approvals when new commits are pushed
- Save changes
References
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels