Skip to content

chore: set CODEOWNERS - #9

Merged
rjan90 merged 1 commit into
masterfrom
chore/set-codeowners
Aug 3, 2026
Merged

chore: set CODEOWNERS#9
rjan90 merged 1 commit into
masterfrom
chore/set-codeowners

Conversation

@BigLep

@BigLep BigLep commented Jul 31, 2026

Copy link
Copy Markdown
Member

@BigLep BigLep self-assigned this Jul 31, 2026
@BigLep
BigLep requested a review from rjan90 July 31, 2026 22:47
@BigLep

BigLep commented Aug 1, 2026

Copy link
Copy Markdown
Member Author

@rvagg @LexLuthr: FYI that you are being made a CODEOWNER in this PR.

Note on CI: the failing check here is pre-existing and unrelated to this change. This PR only adds a .github/CODEOWNERS file, which cannot affect building, testing, linting, or link-checking. Specifically, every CircleCI job fails at the Checkout code step with git@github.com: Permission denied (publickey) / error running git clone ... exit status 128, i.e. CircleCI's deploy key can no longer read the repo.

@BigLep
BigLep marked this pull request as ready for review August 1, 2026 00:13
Copilot AI review requested due to automatic review settings August 1, 2026 00:13

Copilot AI 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.

🟢 Ready to approve

The change is limited to a valid CODEOWNERS addition with no apparent syntax or structural issues.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

Adds a repository-wide GitHub CODEOWNERS configuration so review/approval ownership is automatically applied across the codebase.

Changes:

  • Introduces .github/CODEOWNERS with a catch-all rule (*) assigning ownership to @rvagg and @LexLuthr.
  • Includes a brief comment and reference link describing the source of the ownership decision.
File summaries
File Description
.github/CODEOWNERS Adds default code owners for all paths in the repository.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

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

🤦 needs a migration to GHA, probably just the unified-ci workflows

this change is fine as is though

@rjan90
rjan90 merged commit 2d55fc9 into master Aug 3, 2026
1 of 6 checks passed
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