chore: trim CODEOWNERS entries now enforced org-wide (ENG-2013) - #6
Conversation
|
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
Paths every repo shares (.github/, Dockerfile, Makefile, docker/, alembic/, pyproject.toml, ...) now require a @mindsdb/devops review through the org-level ruleset in mindsdb/terraform, so they leave this file. Anything specific to this repo stays here; the same ruleset enforces CODEOWNERS via require_code_owner_review. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
77feb89 to
7a160d8
Compare
lucas-koontz
left a comment
There was a problem hiding this comment.
fyi: Reviewed the CODEOWNERS change at 7a160d82931a. The active organization baseline covers the removed paths, and /tests/scripts/ remains assigned to DevOps. GitHub reports no CODEOWNERS errors. No blocking findings. The CLA check reports unsigned contributors; skipped deployment jobs do not validate this policy.
What
Edits
.github/CODEOWNERS. The org-level ruleset in mindsdb/terraform (newprod/global/github) requires a@mindsdb/devopsreview on the paths common to all repos, and enforces each repo's CODEOWNERS throughrequire_code_owner_review. So: shared paths leave this file, repo-specific ones stay.Removed:
Kept, as this repo's own rules (enforced by the org ruleset's
require_code_owner_review):Why
One org-wide policy in terraform for what every repo shares; CODEOWNERS for what only this repo needs. Neither duplicates the other.
Targets
main.🤖 Generated with Claude Code