Skip to content

build: upgrade pnpm to v11 and migrate pnpm configuration to pnpm-wor… #33168

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.

Sign up for GitHub

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

Open
alan-agius4 wants to merge 1 commit into
base: main
Choose a base branch
from
+36 −36
Open

build: upgrade pnpm to v11 and migrate pnpm configuration to pnpm-wor… #33168

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 0 additions & 11 deletions .npmrc
View file
Open in desktop

This file was deleted.

5 changes: 2 additions & 3 deletions MODULE.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -131,8 +131,8 @@ use_repo(
pnpm = use_extension("@aspect_rules_js//npm:extensions.bzl", "pnpm")
pnpm.pnpm(
name = "pnpm",
pnpm_version = "10.33.4",
pnpm_version_integrity = "sha512-HGezs1my1AgRm6HtKJ80uPw8aHNBK+xv0mT73IJInlEPy+y5zp0i2ufzt2Jp2EQQRgFL3KU7mXnNelYa1jG4AA==",
pnpm_version = "11.1.0",
pnpm_version_integrity = "sha512-DEToQuVoaywGGoGt2osiWL2IGOlwSyzyxj1WuTGnsukQCS4IUCcAO5bKORGrVqB/bfWrrtK+mSUDTN1oalNbFA==",
)
use_repo(pnpm, "pnpm")

Expand Down Expand Up @@ -172,7 +172,6 @@ npm.npm_translate_lock(
# which would then break execution.
"webdriver-manager": ["local"],
},
npmrc = "//:.npmrc",
pnpm_lock = "//:pnpm-lock.yaml",
Comment thread
alan-agius4 marked this conversation as resolved.
)

Expand Down
24 changes: 2 additions & 22 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,12 @@
"type": "git",
"url": "git+https://github.com/angular/angular-cli.git"
},
"packageManager": "pnpm@10.33.4",
"packageManager": "pnpm@11.1.0",
"engines": {
"node": "^22.22.0 || ^24.13.1 || >=26.0.0",
"npm": "Please use pnpm instead of NPM to install dependencies",
"yarn": "Please use pnpm instead of Yarn to install dependencies",
"pnpm": "10.33.4"
"pnpm": "11.1.0"
},
"author": "Angular Authors",
"license": "MIT",
Expand Down Expand Up @@ -145,25 +145,5 @@
"puppeteer": {
"built": true
}
},
"pnpm": {
"onlyBuiltDependencies": [
"webdriver-manager"
],
"overrides": {
"@angular/build": "workspace:*"
},
"packageExtensions": {
"grpc-gcp": {
"peerDependencies": {
"protobufjs": "*"
}
},
"vitest": {
"peerDependencies": {
"@vitest/coverage-v8": "*"
}
}
}
}
}
32 changes: 32 additions & 0 deletions pnpm-workspace.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,3 +24,35 @@ minimumReleaseAgeExclude:
- '@ngtools/webpack'
- '@schematics/*'
- 'ng-packagr'
overrides:
'@angular/build': workspace:*
packageExtensions:
grpc-gcp:
peerDependencies:
protobufjs: '*'
vitest:
peerDependencies:
'@vitest/coverage-v8': '*'

engineStrict: true
# Disabling pnpm [hoisting](https://pnpm.io/settings#hoist) by setting `hoist:false` is recommended on
# projects using rules_js so that pnpm outside of Bazel lays out a node_modules tree similar to what
# rules_js lays out under Bazel (without a hidden node_modules/.pnpm/node_modules)
hoist: false

# Avoid pnpm auto-installing peer dependencies. We want to be explicit about our versions used
# for peer dependencies, avoiding potential mismatches. In addition, it ensures we can continue
# to rely on peer dependency placeholders substituted via Bazel.
autoInstallPeers: false
allowBuilds:
'@firebase/util': false
'@google/genai': false
'@parcel/watcher': false
bufferutil: false
esbuild: false
lmdb: false
msgpackr-extract: false
protobufjs: false
puppeteer: false
utf-8-validate: false
webdriver-manager: true
Loading