Skip to content

Commit 36bfbd2

Browse files
committed
Document branch protection governance
1 parent cbf6fa0 commit 36bfbd2

4 files changed

Lines changed: 34 additions & 0 deletions

File tree

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,4 @@
1313

1414
- Keep the PR focused.
1515
- Call out any follow-up work explicitly instead of bundling it into this change.
16+
- Do not relax branch protection, required reviews, or required CI checks to merge this PR. Governance changes should be handled separately from feature or docs changes.

AGENTS.md

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# Agent Instructions
2+
3+
This repository is maintained under strict branch protection. Treat those protections as part of the repository policy, not as temporary friction to work around.
4+
5+
## Branch Protection Policy
6+
7+
- Never change branch protection rules, rulesets, required reviews, required status checks, or admin enforcement in order to land a change unless the user explicitly asks for that exact repository-governance change.
8+
- Never disable `CODEOWNERS` review requirements, required approving review counts, required conversation resolution, or required CI checks just to merge a PR.
9+
- Never push directly to a protected branch to bypass the normal review or CI path.
10+
- Never use admin merge, force push, or equivalent bypass mechanisms to land a change unless the user explicitly asks for that governance exception.
11+
- If branch protection blocks a merge, stop and tell the user what requirement is still unmet.
12+
13+
## Preferred Flow
14+
15+
1. Make the change on a feature branch.
16+
2. Open a PR.
17+
3. Let required CI checks complete.
18+
4. Wait for the required review state.
19+
5. Merge only through the normal repository policy path.
20+
21+
## If Blocked
22+
23+
- Report the exact protection rule that is blocking progress.
24+
- Ask the user whether they want to satisfy the rule or intentionally change repository governance.
25+
- Treat repository-governance changes as a separate task from the feature or docs change that triggered the block.

CONTRIBUTING.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ Thanks for contributing to `react-devtool-cli`.
3232
- Add or update tests when behavior changes.
3333
- Preserve the CLI's bias toward deterministic, structured output.
3434
- Do not bundle unrelated refactors into the same PR.
35+
- Do not weaken branch protection, bypass required reviews, or disable required CI checks to land a change. If protection blocks a merge, satisfy the rule or ask the maintainer whether repository governance should change as a separate decision.
3536

3637
## Development
3738

docs/public-repo-strategy.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,3 +51,10 @@ This repository is run as a solo-maintainer open source project. The operating m
5151
- Discussions disabled unless issue volume justifies a separate support channel
5252
- Secret scanning enabled when available
5353
- Topic metadata kept current so the repo is discoverable without overselling scope
54+
55+
## 8. Governance Guardrails
56+
57+
- Branch protection is part of the maintenance policy, not a convenience setting.
58+
- Do not reduce required reviews, required checks, or code owner enforcement just to land an otherwise-ready PR.
59+
- Do not use admin overrides for normal day-to-day maintenance.
60+
- If a branch protection rule becomes counterproductive, change that governance intentionally in a separate maintainer decision, not as part of shipping an unrelated fix.

0 commit comments

Comments
 (0)