Skip to content

chore: point CODEOWNERS at @letta-ai/mods-owners - #81

Merged
cpacker merged 1 commit into
mainfrom
codeowners-mods-owners
Aug 24, 2026
Merged

cpacker merged 1 commit into
mainfrom
codeowners-mods-owners

Conversation

@just-cameron

Copy link
Copy Markdown
Contributor

Summary

  • Collapse the duplicated Charles+Caren CODEOWNERS rules into a single * @letta-ai/mods-owners line so required review is OR, not two individual requests.
  • Team members: @cpacker, @carenthomas, @just-cameron. Any one of them satisfies code-owner review.

GitHub still auto-requests every user listed on a CODEOWNERS line, which is why Charles stayed requested after Caren approved #80. A team is one review request; any member counts.

Needs admin: grant @letta-ai/mods-owners write on this repo. GitHub ignores a team in CODEOWNERS until the team itself has write, even if every member already has it personally. I do not have admin on mods, so I could not do that from here.

Test plan

  • After merge, open a PR that touches any file and confirm the requested reviewer is the mods-owners team, not both Charles and Caren as individuals
  • Confirm one approval from Charles, Caren, or Cameron satisfies the code-owner check

@cpacker
cpacker merged commit 4c249a9 into main Aug 24, 2026
1 check passed
@cpacker
cpacker deleted the codeowners-mods-owners branch August 24, 2026 23:56
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.

2 participants