chore: Update CODEOWNERS for pnpm/uv migration#374
Conversation
There was a problem hiding this comment.
Pull request overview
Updates repository ownership configuration to reflect the migration from npm to pnpm by changing the lockfile referenced in CODEOWNERS.
Changes:
- Replace
package-lock.jsonwithpnpm-lock.yamlinCODEOWNERS. - Touch related file patterns in
CODEOWNERS(no functional code changes).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| pnpm-lock.yaml | ||
| package.json | ||
| .release-please-manifest.json | ||
| CHANGELOG.md | ||
| CHANGELOG.md No newline at end of file |
There was a problem hiding this comment.
CODEOWNERS entries must include at least one owner after the path pattern. As written, pnpm-lock.yaml (and the following file patterns) have no owners, so GitHub will ignore these lines. Either append the intended owner (e.g., @cloudquery/backend) to each entry or remove the redundant lines since * @cloudquery/backend already covers all files.
There was a problem hiding this comment.
Wrong, no owner means anymore can review
There was a problem hiding this comment.
Wrong, no owner means anymore can review
Update CODEOWNERS to reference pnpm-lock.yaml instead of package-lock.json.