Skip to content

Conversation

@nirinchev
Copy link
Collaborator

Proposed changes

An experiment to switch from npm to pnpm.

@nirinchev nirinchev requested a review from a team as a code owner November 21, 2025 15:42
Copilot AI review requested due to automatic review settings November 21, 2025 15:42
Copy link
Contributor

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

This PR migrates the project's package manager from npm to pnpm, updating all scripts, documentation, workflows, and configurations to use pnpm instead of npm.

Key Changes:

  • Switched from npm to pnpm commands throughout the codebase
  • Added packageManager field and pnpm.overrides section to package.json
  • Updated GitHub Actions workflows to use pnpm with proper caching
  • Updated dependency installation commands to use pnpm install --frozen-lockfile

Reviewed changes

Copilot reviewed 13 out of 15 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Added packageManager field, pnpm overrides section, and updated all npm script references to pnpm
scripts/accuracy/runAccuracyTests.sh Replaced npx commands with pnpm dlx and updated comment referencing npm
README.md Added pnpm as a development prerequisite
CONTRIBUTING.md Updated all documentation references from npm to pnpm
Dockerfile Modified to install pnpm before installing the package globally
.github/workflows/*.yml Updated all workflow files to use pnpm/action-setup, pnpm cache, and pnpm commands

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants