Skip to content

chore: include brg8 in code review ownership#2546

Merged
jakehobbs merged 4 commits into
mainfrom
jake/add-brg8-codeowner
Jul 1, 2026
Merged

chore: include brg8 in code review ownership#2546
jakehobbs merged 4 commits into
mainfrom
jake/add-brg8-codeowner

Conversation

@jakehobbs

@jakehobbs jakehobbs commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Summary: Adds @brg8 to the global CODEOWNERS rule so they are included as a code owner across the repository.

Test plan: Not run; this is a CODEOWNERS metadata-only change.

Pull Request Checklist

  • Did you add new tests and confirm existing tests pass? (pnpm test) Not run; not applicable for CODEOWNERS-only metadata.
  • Did you update relevant docs? (docs are found in the docs folder, and guidelines for updating/adding docs can be found in the contribution guide) Not applicable.
  • Do your commits follow the Conventional Commits standard?
  • Does your PR title also follow the Conventional Commits standard?
  • If you have a breaking change, is it correctly reflected in your commit message? No breaking change.
  • Did you run lint (pnpm run lint:check) and fix any issues? (pnpm run lint:write) Not run; not applicable for CODEOWNERS-only metadata.
  • Did you follow the contribution guidelines?

PR-Codex overview

This PR updates the CODEOWNERS file to include additional contributors who will be responsible for code reviews and ownership.

Detailed summary

  • Added the following users to the CODEOWNERS file:
    • @dancoombs
    • @niveda-krish
    • @pavelm
    • @jakehobbs
    • @florrdv
    • @Richard-Dang
    • @avarobinson
    • @blakecduncan
    • @thebrianchen
    • @SahilAujla
    • @CodesMcCabe
    • @brg8

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Co-Authored-By: Codex <noreply@openai.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates repository-wide code review ownership by adding @brg8 to the global CODEOWNERS rule so they’re included as a code owner across the repo.

Changes:

  • Added @brg8 to the * rule in CODEOWNERS.

Comment thread CODEOWNERS Outdated
@@ -1 +1 @@
* @0xfourzerofour @dancoombs @niveda-krish @pavelm @jakehobbs @florrdv @Richard-Dang @clinder777 @avarobinson @blakecduncan @thebrianchen @SahilAujla @CodesMcCabe
* @0xfourzerofour @dancoombs @niveda-krish @pavelm @jakehobbs @florrdv @Richard-Dang @clinder777 @avarobinson @blakecduncan @thebrianchen @SahilAujla @CodesMcCabe @brg8
pavelm
pavelm previously approved these changes Jul 1, 2026
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@jakehobbs
jakehobbs force-pushed the jake/add-brg8-codeowner branch from e55ff65 to f8d34b8 Compare July 1, 2026 19:23
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Richard-Dang
Richard-Dang previously approved these changes Jul 1, 2026
@jakehobbs
jakehobbs enabled auto-merge (squash) July 1, 2026 19:34
Comment thread CODEOWNERS Outdated
@@ -1 +1 @@
* @0xfourzerofour @dancoombs @niveda-krish @pavelm @jakehobbs @florrdv @Richard-Dang @clinder777 @avarobinson @blakecduncan @thebrianchen @SahilAujla @CodesMcCabe
* @0xfourzerofour @dancoombs @niveda-krish @pavelm @jakehobbs @florrdv @Richard-Dang @clinder777 @avarobinson @blakecduncan @thebrianchen @SahilAujla @CodesMcCabe @brg8

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should remove @0xfourzerofour

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done & canaan too

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@jakehobbs
jakehobbs merged commit 05a2b89 into main Jul 1, 2026
9 checks passed
@jakehobbs
jakehobbs deleted the jake/add-brg8-codeowner branch July 1, 2026 19:40
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