Skip to content

pnpm 10 #450

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 3 commits into
base: release
Choose a base branch
from
Draft

pnpm 10 #450

wants to merge 3 commits into from

Conversation

Leksat
Copy link
Member

@Leksat Leksat commented May 14, 2025

pnpm 10 has new deploy mode which works with inject-workspace-packages=true

Currently pnpm deploy ignores the existing lockfile. It does the full resolution step which

  • Takes time
  • Can install dependency versions other than listed in the lockfile

The new deploy solves both issues:

  • There's no resolution step
  • Dependency versions are taken from the lockfile

At the moment pnpm/pnpm#9283 prevents us from switching to pnpm 10.
(There can be more blockers, but we don't know yet.)

Leksat added 2 commits May 14, 2025 13:01
It should not be needed with new pnpm version.
We can't update directly to pnpm 10 because it does not have the migration for the lockfile v6.
gemini-code-assist[bot]

This comment was marked as resolved.

gemini-code-assist[bot]

This comment was marked as off-topic.

@Leksat Leksat force-pushed the test-all/pnpm-10 branch from ff2e45a to 7383b4c Compare May 14, 2025 09:50
pnpm 10 has a new "fast" mode for deploy. This is the main reason for the upgrade.
@Leksat Leksat force-pushed the test-all/pnpm-10 branch from 7383b4c to 07d7cd1 Compare May 16, 2025 08:21
@Leksat Leksat marked this pull request as draft May 16, 2025 08:28
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.

1 participant