Skip to content

chore(github): add CODEOWNERS#2067

Open
UnbornAztecKing wants to merge 1 commit intomainfrom
security-audit-codeowners
Open

chore(github): add CODEOWNERS#2067
UnbornAztecKing wants to merge 1 commit intomainfrom
security-audit-codeowners

Conversation

@UnbornAztecKing
Copy link
Contributor

Summary

  • Introduces CODEOWNERS to standardize review ownership and auto-assign reviewers.
  • Sets default ownership to @dydxprotocol/web-code-owners for all files.
  • Assigns @dydxprotocol/security as owners for /.github/ and /CODEOWNERS to enforce security oversight on governance files.

Details

  • Repo governance:
    • GitHub will auto-request reviews from code owners; with branch protection “Require review from Code Owners,” merges may require approvals from these teams.
  • Path ownership:
    • "*" → @dydxprotocol/web-code-owners (default).
    • "/.github/" → @dydxprotocol/security (workflows, templates, policies).
    • "/CODEOWNERS" → @dydxprotocol/security (self-governance of ownership rules).

Risk & Impact

  • Low risk: configuration-only; no runtime behavior changes.
  • Potential reviewer routing changes; may block merges if branch protection requires code owner approvals.

Testing

  • No tests applicable; config addition only.

Reviewer Notes

  • Verify the teams exist and have correct permissions: @dydxprotocol/web-code-owners, @dydxprotocol/security.
  • Confirm the intended scope of "/.github/" (includes all workflow and policy files).
  • If required, ensure branch protection rules are configured to require code owner reviews.

@UnbornAztecKing UnbornAztecKing requested a review from a team as a code owner January 30, 2026 23:56
@vercel
Copy link

vercel bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v4-staging Ready Ready Preview, Comment Jan 31, 2026 0:51am
v4-testnet Ready Ready Preview, Comment Jan 31, 2026 0:51am

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants