Skip to content

chore: add CODEOWNERS so the code-owner review rule has effect - #1

Closed
AndreaSucreG wants to merge 2 commits into
mainfrom
chore/add-codeowners
Closed

chore: add CODEOWNERS so the code-owner review rule has effect#1
AndreaSucreG wants to merge 2 commits into
mainfrom
chore/add-codeowners

Conversation

@AndreaSucreG

@AndreaSucreG AndreaSucreG commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

The main ruleset sets require_code_owner_review: true, but there was no
CODEOWNERS file anywhere (root, .github/, docs/) — so that rule was silently
a no-op and the protection config overstated what it actually enforced.

This adds .github/CODEOWNERS making @AndreaSucreG the sole owner of *.
Every change to the repo therefore needs her approval.

Known consequence: GitHub does not accept an author's own code-ownership as
satisfying the code-owner review requirement. With a single owner and no bypass
actors, PRs opened by @AndreaSucreG cannot be merged — including this one. To
land it, either grant repo-admin bypass on the main ruleset temporarily, or
have @felixcarpena / @raquelyuste approve it while CODEOWNERS is not yet in
effect on main (the file only starts applying once merged).

🤖 Generated with Claude Code


Note

Low Risk
Repository governance only; no application runtime or data-path changes.

Overview
Adds .github/CODEOWNERS with a * rule pointing at @AndreaSucreG, so pull requests are no longer able to satisfy branch protection’s code-owner review requirement with an empty owner list.

This closes the gap where require_code_owner_review: true on main had no effect because no CODEOWNERS file existed. After merge, every change needs an approval from that owner; the PR notes that GitHub won’t count the author’s own review as the required code-owner review when they are the sole owner.

Reviewed by Cursor Bugbot for commit f2b7600. Bugbot is set up for automated code reviews on this repo. Configure here.

@AndreaSucreG AndreaSucreG reopened this Aug 3, 2026
@AndreaSucreG
AndreaSucreG marked this pull request as ready for review August 3, 2026 13:19
@AndreaSucreG

Copy link
Copy Markdown
Collaborator Author

@AndreaSucreG
AndreaSucreG deleted the chore/add-codeowners branch August 5, 2026 17:54
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.

2 participants