chore: remove CODEOWNERS entries now enforced org-wide (ENG-2013) - #89
chore: remove CODEOWNERS entries now enforced org-wide (ENG-2013)#89hamishfagg wants to merge 1 commit into
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>
89ced64 to
3ef75c0
Compare
hamishfagg
changed the title
lucas-koontz
left a comment
lucas-koontz
left a comment
There was a problem hiding this comment.
fyi: Reviewed the CODEOWNERS deletion and effective rules on main. The live organization baseline already requires DevOps approval for .github changes, so this removes duplicate ownership. Unit tests passed. The CLA check requires a contributor signature; that result is unrelated to this deletion.
2 participants
What
Deletes
.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:
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