Skip to content

Document the maintainer bypass of the gate - #573

Closed
jserv wants to merge 2 commits into
mainfrom
doc-admin-bypass
Closed

jserv wants to merge 2 commits into
mainfrom
doc-admin-bypass

Conversation

@jserv

@jserv jserv commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

The main review gate ruleset lets the repository admin bypass the review and the required checks from a pull request, but docs/development.md said administrators may not bypass, and that the most recent push needs approval, which the ruleset does not require. This states the settings that are actually in force, and says that the gate binds contributors and not the maintainer. Documentation only.

Interactive review: not required
Head: 2676d10
Base: d6b90af
Diff: sha256:99c89af49556198cda056a5eb2bf8750892e879e20fa6367c1170bff40db33f6
PR text lines: 10; commits at 200 lines or more: 0

The ruleset lets the repository admin bypass the review and the
required checks from a pull request, so the gate binds contributors
and not the maintainer. The documentation said administrators may not
bypass. State the setting that is actually in force.

Change-Id: I5b39fc82e900440ad64c3e5e6827acd2f0a3851d
cubic-dev-ai[bot]

This comment was marked as resolved.

The ruleset does not require approval of the most recent push, and a
sole maintainer who authors a pull request could not satisfy it. State
the rules that are in force.

Change-Id: I37be8f724e77e65b164c1769acfa015ff710d1d8
@jserv

jserv commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #574, which carries this fix as its first commit.

@jserv jserv closed this Nov 2, 2026
@jserv
jserv deleted the doc-admin-bypass branch November 2, 2026 19:21
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