chore(CODEOWNERS): remove blanket-ownership file#1989
Merged
Conversation
The previous CODEOWNERS assigned every path in the repo to all three maintainers (@woile @Lee-W @noirbizarre), so every PR pinged all three for review regardless of scope. Recent commit history shows ownership in practice is much more concentrated and uneven, and the broad rule produces review-request spam without reflecting who actually maintains each area. Removing the file lets GitHub fall back to repository-admin assignment without forcing a review request on three people for every change. A more granular per-area CODEOWNERS can be reintroduced later if maintainers want one. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Lee-W
approved these changes
May 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Remove the repo-wide
CODEOWNERSrule that assigned every path to all three maintainers.Because GitHub treats CODEOWNERS owners as required (or at least auto-requested) reviewers, this single line caused every PR -- regardless of scope -- to request review from all three maintainers. That's review-request spam and doesn't reflect who actually maintains each area today.
Why now / data behind the change
I ran
git logper subtree over the last two years to see who actually touches each area. Highlights (commit counts;Yu-Ting HsiungandTim Hsiungare the same person -- @bearomorphism -- confirmed by commit email):commitizen/corecommitizen/commands/commitizen/providers/commitizen/changelog_formats/commitizen/cz/,commitizen/config/docs/tests/.github/, CI,pyproject.tomlTwo observations:
*rule with three names is the worst-case configuration: it maximises notification spam while providing no routing signal about who is best placed to review.What I considered instead
A per-area CODEOWNERS (e.g.
commitizen/providers/ @X,.github/ @Y,docs/ @Z) was the obvious alternative. I'm not proposing that here for two reasons:Once maintainers agree on per-area primary owners, a more granular CODEOWNERS can be reintroduced in a follow-up.
Checklist
Was generative AI tooling used to co-author this PR?
Generated-by: GitHub Copilot CLI following the guidelines
Code Changes
.github/governance file is removed; no code paths are affected.uv run poe alllocally to ensure this change passes linter check and tests -- N/A for a single-file deletion in.github/; pre-commit hooks ran clean on the commit.git show --stat.Documentation Changes
N/A.
Expected Behavior
After merge, new PRs will no longer auto-request review from
@woile,@Lee-W, and@noirbizarrepurely on the basis of the CODEOWNERS rule. Maintainers can still self-assign or be requested by the PR author as appropriate.Steps to Test This Pull Request
.github/CODEOWNERSis gone on this branch.masterand confirm no automatic reviewer request is added by GitHub's CODEOWNERS handling.Suggested complement: auto-request Copilot code review
To replace the always-on review request without losing a default first-pass reviewer, consider enabling GitHub's built-in auto Copilot code review on this repo. A maintainer with admin access can do it via:
Settings -> Rules -> Rulesets -> New branch ruleset-> targetmaster(and any release branches) -> tick "Automatically request Copilot code review".Useful sub-options:
Reference: https://docs.github.com/en/copilot/how-tos/use-copilot-agents/request-a-code-review/configure-automatic-review.
Notes:
commitizen-toolsorg to be on a Copilot plan that covers code review (Business / Enterprise), or PR authors to have Copilot Pro/Pro+. If neither applies, this option simply won't appear in the ruleset UI.The pairing -- drop the blanket CODEOWNERS rule, add the Copilot ruleset -- gives every PR an always-on first review without three human maintainers being requested by default.
Additional Context
No related issue. Happy to follow up with a per-area CODEOWNERS once @woile / @Lee-W / @noirbizarre agree on who should be the default reviewer for which subtree.