Skip to content

Clarify administrator branch protection bypass - #96

Merged
arjitj2 merged 2 commits into
mainfrom
arjit/admin-bypass-docs
Sep 28, 2026
Merged

arjitj2 merged 2 commits into
mainfrom
arjit/admin-bypass-docs

Conversation

@arjitj2

@arjitj2 arjitj2 commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

Closes #95.

Document the administrator bypass requested by the repository owner. Administrators may push directly to main or merge without first updating a PR branch from main. Non-admin contributors still require a PR, passing GitHub Actions verify check, and an up-to-date branch. The Allow force pushes and Allow deletions settings remain disabled.

Validation:

  • Changed only the live enforce_admins setting from true to false through GitHub's branch-protection API. Readback and before/after comparison confirm every other protection field is unchanged. No additional rulesets apply.
  • Documentation checker, 16 documentation tests, whitespace check, and plugin-tree equality pass.
  • Only CONTRIBUTING.md changes. Previously passing package tests, typecheck, static checks and plugin validation still apply to these identical plugin bytes. CI repeats the required checks for this head.

No test push to main was needed. The setting is confirmed through GitHub's API and documented admin-bypass semantics.

Copilot review was requested and returned a quota-limit notice without code findings.

@arjitj2
arjitj2 requested a lite review from Copilot September 28, 2026 20:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@arjitj2

arjitj2 commented Sep 28, 2026

Copy link
Copy Markdown
Owner Author

Independent maintenance review: PASS for 774ab7a against 928839a.

The guide describes the requested administrator bypass and names the unchanged Allow force pushes and Allow deletions settings. Saved API readback shows only enforce_admins changed from true to false. The earlier wording ambiguity is resolved. Only CONTRIBUTING.md changes; the packaged plugin tree remains identical. Documentation and whitespace checks pass in a clean detached checkout. No remaining findings.

@arjitj2
arjitj2 merged commit 09b71ac into main Sep 28, 2026
1 check passed
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.

Allow admin bypass of main branch requirements

2 participants