Skip to content

chore: Update CODEOWNERS for pnpm/uv migration#144

Merged
kodiakhq[bot] merged 1 commit intomainfrom
chore/update-codeowners-pnpm-uv
Apr 1, 2026
Merged

chore: Update CODEOWNERS for pnpm/uv migration#144
kodiakhq[bot] merged 1 commit intomainfrom
chore/update-codeowners-pnpm-uv

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Update CODEOWNERS to reference pnpm-lock.yaml instead of package-lock.json.

Copilot AI review requested due to automatic review settings April 1, 2026 13:25
@erezrokah erezrokah added the automerge Add to automerge PRs once requirements are met label Apr 1, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates repository CODEOWNERS entries to reflect the move from npm’s package-lock.json to pnpm-lock.yaml.

Changes:

  • Replace package-lock.json reference with pnpm-lock.yaml in CODEOWNERS.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

* @cloudquery/cloudquery-framework

package-lock.json
pnpm-lock.yaml
Copy link

Copilot AI Apr 1, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pnpm-lock.yaml is listed without any owners. In CODEOWNERS, each pattern line must include one or more owners (otherwise GitHub treats the line as invalid/ignored), so this entry currently has no effect. Add the intended owner(s) on this line (and consider anchoring it as /pnpm-lock.yaml if you only mean the repo-root lockfile).

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong, no owner means anymore can review

@kodiakhq kodiakhq bot merged commit 02d51e7 into main Apr 1, 2026
10 checks passed
@kodiakhq kodiakhq bot deleted the chore/update-codeowners-pnpm-uv branch April 1, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Add to automerge PRs once requirements are met

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants