Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Greptile SummaryThis PR adds the Key changes:
To fix: Consolidate both teams onto a single line: Confidence Score: 2/5
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Pull Request opened] --> B{Match CODEOWNERS rules}
B --> C["Rule 1: * @workos/typescript"]
B --> D["Rule 2: * @workos/dx (last match)"]
C -->|Overridden| E[❌ @workos/typescript NOT notified]
D -->|Last rule wins| F[✅ @workos/dx notified]
G["Intended fix: * @workos/typescript @workos/dx"] --> H[✅ @workos/typescript notified]
G --> I[✅ @workos/dx notified]
Last reviewed commit: 86856e9 |
.github/CODEOWNERS
Outdated
| * @workos/typescript | ||
|
|
||
| # DX Team | ||
| * @workos/dx |
There was a problem hiding this comment.
Last matching rule overrides earlier rules in CODEOWNERS
In GitHub's CODEOWNERS format, when multiple patterns match the same file, the last matching rule wins. Because * @workos/dx appears after * @workos/typescript, the DX team replaces the TypeScript team as codeowners for all files — rather than adding them alongside. This means pull requests will only request a review from @workos/dx, not @workos/typescript.
To make both teams co-owners, they should be listed together on a single line:
| * @workos/dx | |
| * @workos/typescript @workos/dx |
This is the standard way to assign multiple teams as codeowners for the same glob pattern. See GitHub docs on CODEOWNERS.
Summary
@workos/dxteam as code owners for the repository🤖 Generated with Claude Code