Skip to content

Add CODEOWNERS - #42

Merged
joshsiaw merged 1 commit into
mainfrom
cato/add-codeowners
Oct 9, 2026
Merged

joshsiaw merged 1 commit into
mainfrom
cato/add-codeowners

Conversation

@joshsiaw

@joshsiaw joshsiaw commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

The main ruleset already requires a review from a code owner, but with no CODEOWNERS file that rule had nothing to enforce. This adds .github/CODEOWNERS, making both maintainers owners of the whole repository:

* @joshsiaw @maciej-szlosarczyk

Once this merges, every PR needs an approval from one of them. The ruleset also requires the approval to come from someone other than whoever pushed last, so each maintainer's PRs need the other's approval.

CONTRIBUTING.md now says the approval must come from a code owner.

GitHub's CODEOWNERS validation reports no errors on this branch.

🤖 Generated with Claude Code

The main ruleset already requires a code owner's review, but with no
CODEOWNERS file that rule had nothing to enforce. Both maintainers now
own the whole repository, and CONTRIBUTING says the approval must come
from one of them.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@joshsiaw
joshsiaw added this pull request to the merge queue Oct 9, 2026
Merged via the queue into main with commit de6d5b4 Oct 9, 2026
5 checks passed
@joshsiaw
joshsiaw deleted the cato/add-codeowners branch October 9, 2026 08:51
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