chore: Add CODEOWNERS file to better reflect ownership#681
chore: Add CODEOWNERS file to better reflect ownership#681SebastienPoitras wants to merge 2 commits intomasterfrom
Conversation
Greptile SummaryThis PR refines
Confidence Score: 4/5Safe to merge only if the companion ruleset enforcing engineering review is merged simultaneously; merging this file alone leaves several packages without any required reviewer. The removal of the CODEOWNERS — the missing catch-all and unowned packages need to be resolved. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
PR[Pull Request / File Change]
PR --> CO{CODEOWNERS match?}
CO -->|Explicit match| PKG[Package-specific owner notified]
CO -->|No match - BEFORE this PR| ENG[botpress/engineering notified via wildcard *]
CO -->|No match - AFTER this PR| NONE[❌ No owner - no required review]
PKG --> E1[entities → @franklevasseur]
PKG --> E2[depsynky / es-node / jex / log4bot / opapi / readiness / verel / yargs-extra / slackdown → @botpress/squid]
PKG --> E3[pingrip / tunnel → @botpress/kraken]
PKG --> E4[promex / trail → @michaelmass]
PKG --> E5[ptb-schema → @franklevasseur]
NONE --> GAP["Unowned: expresso, genenv, mcp-gen,\nretry, signature-verification, thicktoken, zui\n+ root files"]
style NONE fill:#f55,color:#fff
style GAP fill:#faa
Reviews (1): Last reviewed commit: "chore: Add CODEOWNERS file to better ref..." | Re-trigger Greptile |
A ruleset will also be added to enforce a minimum of one review by engineering, ensuring coverage of all files in the repository.