Skip to content

feat: add evidence-locker-mvp-reviewers team - #204

Open
beatrizmcouto wants to merge 2 commits into
complytime:mainfrom
beatrizmcouto:feat/evidence-locker-mvp-reviewers
Open

feat: add evidence-locker-mvp-reviewers team#204
beatrizmcouto wants to merge 2 commits into
complytime:mainfrom
beatrizmcouto:feat/evidence-locker-mvp-reviewers

Conversation

@beatrizmcouto

Copy link
Copy Markdown
Contributor

Summary

  • Add a closed GitHub team evidence-locker-mvp-reviewers with @hbraswelrh, @sedonnel, and @trevor-vaughan.
  • This is the membership source of truth for Evidence Locker MVP review assignment. Change the members list here to add or remove reviewers over time.
  • CODEOWNERS cannot filter on milestones, so assignment itself is handled by companion org-infra PR feat: assign Evidence Locker MVP reviewers from peribolos team org-infra#539.

Related Issues

None.

Review Hints

  • Merge this before the org-infra workflow PR so Peribolos creates the team first.
  • After merge, Peribolos apply (push to main or the daily 05:30 UTC job) creates the team in GitHub.
  • The team has write on the same code repos as complytime-dev so members can be requested as reviewers.

Made with Cursor

Create a closed GitHub team so Evidence Locker MVP reviewer membership
can change in peribolos.yaml without editing CODEOWNERS.

Co-authored-by: Cursor <cursoragent@cursor.com>

@sedonnel sedonnel 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.

LGTM

Comment thread peribolos.yaml
- sedonnel
- trevor-vaughan
repos:
community: write

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question
@beatrizmcouto Don't we need nunya in this list?

@beatrizmcouto beatrizmcouto Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes — good catch. The Evidence Locker MVP milestone lives on nunya, so this team needs that repo or GitHub cannot request these people as reviewers there.

Added nunya: maintain in 69c917f (maintain so members can create sub-issues; GitHub hides that for Read). The secret nunya-access team stays unmanaged by Peribolos (--ignore-secret-teams).

Note: #205 (nunya on complytime-dev) was closed because @jpower432 is handling that access another way. This team still needs the grant because assignment for this milestone is on nunya.

@hbraswelrh hbraswelrh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Evidence Locker MVP work lives in nunya, so reviewers need repo access
to be requested and to create sub-issues.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

4 participants