Skip to content

fix(codeowners): substitute the un-instantiated {{OWNER}} placeholder - #50

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/codeowners-substitute-owner
Jul 29, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
fix/codeowners-substitute-owner

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

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 with require_code_owner_review becomes unsatisfiable.

just init does 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

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>
@gitar-bot

gitar-bot Bot commented Jul 29, 2026 •

Copy link
Copy Markdown

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.
Learn more

Code Review ✅ Approved

Substitutes the un-instantiated {{OWNER}} placeholder in the CODEOWNERS file with the correct team handle to ensure valid code ownership rules. No issues found.

Auto-approved and auto-merge armed: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria. — merges when pipeline and required approvals pass.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot

gitar-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

⚠️ Gitar auto-approved this PR but could not enable auto-merge: auto-merge is disabled for this repository — enable "Allow auto-merge" in the repository settings.

@gitar-bot gitar-bot Bot 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.

Gitar has auto-approved this PR and enabled auto-merge (configure)

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Jul 29, 2026
@hyperpolymath
hyperpolymath merged commit 4ad8808 into main Jul 29, 2026
22 of 25 checks passed
@hyperpolymath
hyperpolymath deleted the fix/codeowners-substitute-owner branch July 29, 2026 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant