Skip to content

docs(maintenance): document the ruleset bypass policy - #62

Merged
laurenhitchon merged 1 commit into
mainfrom
chore/document-ruleset-bypass-policy
Jul 29, 2026
Merged

docs(maintenance): document the ruleset bypass policy#62
laurenhitchon merged 1 commit into
mainfrom
chore/document-ruleset-bypass-policy

Conversation

@laurenhitchon

Copy link
Copy Markdown
Contributor

Documents the fleet ruleset bypass policy in MAINTENANCE.md following the 2026-07-29 removal of the admin always-on bypass from all 24 repos' Protect main rulesets.

  • Only bypass actor everywhere is now the per-repo release deploy key.
  • Rationale: an admin direct push skipped the required install / install gate silently (demonstrated on nswds-ui).
  • Emergency path documented: temporarily disable ruleset enforcement, push, re-enable.
  • New automations should get a dedicated deploy key, not a role/team bypass.

MAINTENANCE.md is not in the sync map — no fleet fan-out.

The fleet rulesets' only bypass actor is now the per-repo release deploy
key. The admin always-on bypass from the original rollout was removed
across all 24 repos on 2026-07-29 — an admin push to main skipped the
install / install gate silently. Documents the emergency path (temporary
enforcement disable) in its place.
Copilot AI review requested due to automatic review settings July 29, 2026 05:18
@DFSIdigital

DFSIdigital commented Jul 29, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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.

Pull request overview

Documents the updated fleet “Protect main” ruleset bypass policy in MAINTENANCE.md following the removal of the always-on admin bypass, clarifying who can bypass required CI gates and how to handle emergencies.

Changes:

  • Adds a “Ruleset bypass policy” section stating the release deploy key is the sole bypass actor.
  • Explains the rationale (admin direct pushes can silently bypass required gates).
  • Documents an emergency procedure (temporarily disable enforcement, push, re-enable) and guidance for future automations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread MAINTENANCE.md
Comment on lines +123 to +125
enforcement to `disabled` (repo Settings → Rules, or
`gh api -X PUT repos/digitalnsw/<repo>/rulesets/<id>` resending the full
definition with `"enforcement": "disabled"`), push, and re-enable — a
@laurenhitchon
laurenhitchon merged commit c55ae62 into main Jul 29, 2026
10 checks passed
@laurenhitchon
laurenhitchon deleted the chore/document-ruleset-bypass-policy branch July 29, 2026 05:22
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.

3 participants