Skip to content

README: main's protection is enforced — code-owner review is on; drop the stale 'known gap' - #10

Merged
virtualsteve-exa merged 1 commit into
mainfrom
docs/main-protection-current
Sep 14, 2026
Merged

virtualsteve-exa merged 1 commit into
mainfrom
docs/main-protection-current

Conversation

@virtualsteve-exa

Copy link
Copy Markdown
Collaborator

The "For maintainers" section still carried a Known gap paragraph saying .github/CODEOWNERS only auto-requests reviewers "until Require review from Code Owners is enabled on the branch protection rule", and telling readers to treat a green catalog check as evidence rather than proof.

That state no longer exists. The branch protection on main, read from the API today:

Setting Value
Approving reviews required 1
Require review from code owners on
Dismiss stale approvals on new pushes on
Required check catalog, strict (branch up to date with main)
Force pushes / deletion blocked

CODEOWNERS covers /.claude-plugin/, /scripts/ and /.github/. So a PR that edits the workflow or the validator cannot merge without a code owner reading that diff, which closes exactly the gap the paragraph described. The bullet now states what is enforced, in one place, and the caveat is gone.

Docs only; no change to the catalog or the rules.

🤖 Generated with Claude Code

https://claude.ai/code/session_016Q9UohWfy2scPhr4pavhqd

…'known gap' paragraph described a state that no longer exists

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016Q9UohWfy2scPhr4pavhqd
@virtualsteve-exa
virtualsteve-exa merged commit 1444db2 into main Sep 14, 2026
1 check passed
@virtualsteve-exa
virtualsteve-exa deleted the docs/main-protection-current branch September 14, 2026 05:08
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