Skip to content

chore: add CODEOWNERS - #134

Merged
ben-miru merged 1 commit into
mainfrom
chore/codeowners
Oct 21, 2026
Merged

ben-miru merged 1 commit into
mainfrom
chore/codeowners

Conversation

@bens-agent

@bens-agent bens-agent commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Requested by bens-agent · project thread

Summary

Adds .github/CODEOWNERS making @ben-miru the code owner of every path in this repo.

This is step one of requiring a human approval on every PR. A follow-up PR in mirurobotics/infra (github/terraform/repos.tf) turns on 1 required approval, "require review from Code Owners", and "require approval of the most recent push". Until that lands, this file only auto-requests reviews.

Precedent

No repo in the org has a CODEOWNERS file yet; all repos get the same one-line format.


Generated by Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DHfvjCLNmabAiGhEsWgBks
@github-actions

Copy link
Copy Markdown

AI review: approve for 56edea8

This PR adds only .github/CODEOWNERS, which makes @ben-miru the code owner of every path (*). The syntax is valid, and no other CODEOWNERS file exists in the repo root or docs/ that would take precedence.

One change-management note: the file's comment says a code owner "must approve every PR to main". That is only true once the follow-up branch-protection change in mirurobotics/infra lands. Until then the file just auto-requests reviews.

When that change lands, every PR will need @ben-miru's approval, so they are a single point of failure for merges. Bot-authored PRs such as Dependabot updates will be gated too. Consider adding a backup owner or a team. This doesn't block the PR.

Review run

@ben-miru
ben-miru merged commit a076ebb into main Oct 21, 2026
29 checks passed
@ben-miru
ben-miru deleted the chore/codeowners branch October 21, 2026 00:05
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.

3 participants