Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enforce review policy #9120 #102

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

mhitza
Copy link
Collaborator

@mhitza mhitza commented Nov 20, 2024

Types of changes

  • feat: non-breaking change which adds new functionality
  • chore: general tasks or anything that doesn't fit the other commit types

Checklist

@mhitza mhitza changed the title Review policy template 9120 Enforce review policy #9120 Nov 20, 2024
@mhitza mhitza marked this pull request as draft November 20, 2024 16:32
@mhitza mhitza force-pushed the review-policy-template-9120 branch from 41384c3 to ad7923b Compare December 27, 2024 15:24
Having this flag as an opt-in prevents situations where a developer
working individually on a repository locks himself out from merging PRs
and pushing to the master branch without going through additional hoops.
One controls the pull request review policy, and the second one prevents
collaborators from pushing directly to the default branch (prevents
commit merging as well, only doable by admins)
@mhitza mhitza force-pushed the review-policy-template-9120 branch from ad7923b to 9758bc1 Compare January 19, 2025 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant