Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Greptile SummaryThis PR adds Key Finding: The two separate Confidence Score: 2/5
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["PR opened on workos-php"] --> B["GitHub evaluates CODEOWNERS"]
B --> C["Rule 1: * @workos/php"]
C --> D["Rule 2: * @workos/dx ← last match wins"]
D --> E["Required reviewer: @workos/dx ONLY"]
E --> F["@workos/php silently dropped ⚠️"]
style F fill:#f88,stroke:#c00,color:#000
style D fill:#ffa,stroke:#aa0,color:#000
Last reviewed commit: 2d1f497 |
.github/CODEOWNERS
Outdated
| * @workos/php | ||
|
|
||
| # DX Team | ||
| * @workos/dx |
There was a problem hiding this comment.
GitHub's CODEOWNERS file uses a last-match-wins strategy — the last matching pattern takes precedence over earlier ones. By adding a second * @workos/dx rule, the @workos/php team is effectively removed as a required reviewer for all files, because @workos/dx now matches last.
If the intent is to require both teams to review, both owners should be listed on the same line:
| * @workos/php | |
| # DX Team | |
| * @workos/dx | |
| * @workos/php @workos/dx |
Reference: GitHub docs on CODEOWNERS
Summary
@workos/dxteam as code owners for the repository🤖 Generated with Claude Code