Skip to content

Add branch protection ruleset requiring PRs on master/main/dev #10

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.

Sign up for GitHub

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

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

Add branch protection ruleset requiring PRs on master/main/dev #10

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fd2ccf2
Initial plan
Copilot Mar 1, 2026
e70c76a
Add branch protection ruleset requiring pull requests on master/main/dev
Copilot Mar 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 30 additions & 0 deletions .github/rulesets/require-pull-request.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
{
"name": "Require pull request on master/main",
"target": "branch",
"enforcement": "active",
"conditions": {
"ref_name": {
"include": ["refs/heads/master", "refs/heads/main", "refs/heads/dev"],
"exclude": []
}
},
"bypass_actors": [
{
"actor_type": "RepositoryRole",
"actor_id": 5,
"bypass_mode": "always"
}
],
"rules": [
{
"type": "pull_request",
"parameters": {
"required_approving_review_count": 0,
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚩 required_approving_review_count is 0 — PRs required but no reviews enforced

Setting required_approving_review_count to 0 means a pull request is required to merge into the protected branches, but it can be merged without any approving review. Combined with dismiss_stale_reviews_on_push: false, require_code_owner_review: false, and require_last_push_approval: false, the ruleset only enforces that changes go through a PR — not that anyone actually reviews them. This may be intentional (the CONTRIBUTING.md mentions the repo is relatively new and the PR expectations are lightweight), but it's worth confirming this is the desired protection level, especially for main/master.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

"dismiss_stale_reviews_on_push": false,
"require_code_owner_review": false,
"require_last_push_approval": false,
"required_review_thread_resolution": false
}
}
]
}