Skip to content

chore: file three managed-repository reports and fix the main review wording - #1096

Merged
REPPL merged 2 commits into
mainfrom
chore/inbox-triage-1018
Oct 18, 2026
Merged

REPPL merged 2 commits into
mainfrom
chore/inbox-triage-1018

Conversation

@REPPL

@REPPL REPPL commented Oct 18, 2026

Copy link
Copy Markdown
Collaborator

Files the three reports managed repositories sent back to abcd's inbox, and fixes the one that named a wording fault.

  • The main review rule. A managed repository found that its copy of the "main review" ruleset let the owner's own pull request merge with no approval, and asked whether the rule holds anyone. GitHub's ruleset documentation requires a code owner's approval on a pull request that modifies owned content, whatever the approval count, so the rule binds every writer but a repository admin, and the owner's pull requests pass by the admin bypass. The .github/CODEOWNERS header and .abcd/work/rulesets/README.md now say so.
  • A staged go-public flow (enhancement), filed at minor rather than the sender's major: it reports no defect, and a major would hold every release cut until the feature ships. The reason is recorded as a decision.
  • A design verb (enhancement, future-work seed), filed as sent.

Resolves: iss-2610180913341869
Refs: iss-2610180913363115
Refs: iss-2610180913376757
Refs: dcn-2610180914543900

Assisted-by: Claude:claude-opus-5-5

REPPL added 2 commits October 18, 2026 10:15
A staged go-public flow and a design verb, both enhancement requests
filed back by managed repositories. The go-public one is filed at minor,
not the sender's major, because it reports no defect and a major would
hold every release cut until the feature ships.

Refs: iss-2610180913363115, iss-2610180913376757
Refs: dcn-2610180914543900
Assisted-by: Claude:claude-opus-5-5
A managed repository reported that the copied "main review" ruleset let
the owner's own pull request merge with no approval, and asked whether it
holds anyone. GitHub requires a code owner's approval on owned content
whatever the approval count, so it binds every writer but an admin, and
the owner's pull requests pass by the admin bypass. The CODEOWNERS header
and the rulesets README now say so.

Resolves: iss-2610180913341869
Assisted-by: Claude:claude-opus-5-5
@REPPL
REPPL enabled auto-merge October 18, 2026 11:49
@REPPL
REPPL added this pull request to the merge queue Oct 18, 2026
Merged via the queue into main with commit c198978 Oct 18, 2026
14 checks passed
@REPPL
REPPL deleted the chore/inbox-triage-1018 branch October 18, 2026 12:52
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