fix(codeowners): substitute the un-instantiated {{OWNER}} placeholder - #58
Conversation
CODEOWNERS carried the literal `@{{OWNER}}`. GitHub's own validator
(/repos/{owner}/{repo}/codeowners/errors) reports every such rule as
'Invalid owner', which means the file assigns NO code owners at all -- and any
ruleset with require_code_owner_review becomes unsatisfiable, since a review
from a non-existent owner can never arrive.
This is not a template artefact here: `just init` performs a repo-wide brace-token
substitution and simply never ran for this repo. Measured estate-wide 2026-07-29:
54 repos carried the placeholder, of which 2 are genuine templates where it is
CORRECT and are deliberately untouched.
Substituted with the repo's actual owner (hyperpolymath).
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Note Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime. Code Review ✅ ApprovedSubstitutes the un-instantiated {{OWNER}} placeholder in the CODEOWNERS file to ensure valid ownership rules and prevent unsatisfiable branch protection requirements. No issues found.
OptionsDisplay: compact → Showing less information. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
|
|
CODEOWNERS carried the literal
@{{OWNER}}. GitHub's own validator reports every such rule as Invalid owner, so the file assigns no code owners at all — and any ruleset withrequire_code_owner_reviewbecomes unsatisfiable.just initdoes a repo-wide brace-token substitution and simply never ran here. Measured estate-wide: 54 repos carried it; 2 are genuine templates where the placeholder is correct and are deliberately untouched.🤖 Generated with Claude Code