Skip to content

Commit

Permalink
Fix pnpm ERR_INVALID_THIS on Node 20 (#51406)
Browse files Browse the repository at this point in the history
### What?

Fixes pnpm/pnpm#6424

### Why?

So users of Node 20 can contribute to Next.js

### How?

Upgrades pnpm with back compatibility to patch `ERR_INVALID_THIS` error.
If Node 14 support isn't necessary, I'd recommend upgrading to pnpm 8
for continued feature support and committing the lockfile.

---------

Co-authored-by: JJ Kasper <jj@jjsweb.site>
  • Loading branch information
nickmccurdy and ijjk committed Jun 16, 2023
1 parent d695298 commit 1602c2a
Show file tree
Hide file tree
Showing 9 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/actions/next-stats-action/Dockerfile
Expand Up @@ -14,7 +14,7 @@ RUN ln $(which python3) /usr/bin/python
RUN curl -sfLS https://install-node.vercel.app/v18 | bash -s -- -f

# Install node_modules
RUN npm i -g pnpm@7.24.3 yarn@1.22.19
RUN npm i -g pnpm@7.32.2 yarn@1.22.19
RUN cd /next-stats && pnpm install --production

RUN git config --global user.email 'stats@localhost'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/build_and_deploy.yml
Expand Up @@ -8,7 +8,7 @@ on:
env:
NAPI_CLI_VERSION: 2.14.7
TURBO_VERSION: 1.9.6
PNPM_VERSION: 7.24.3
PNPM_VERSION: 7.32.2
NODE_MAINTENANCE_VERSION: 16
NODE_LTS_VERSION: 18

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/build_and_test.yml
Expand Up @@ -9,7 +9,7 @@ on:
env:
NAPI_CLI_VERSION: 2.14.7
TURBO_VERSION: 1.9.6
PNPM_VERSION: 7.24.3
PNPM_VERSION: 7.32.2
NODE_MAINTENANCE_VERSION: 16
NODE_LTS_VERSION: 18
TEST_CONCURRENCY: 6
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/build_reusable.yml
Expand Up @@ -47,7 +47,7 @@ on:
env:
NAPI_CLI_VERSION: 2.14.7
TURBO_VERSION: 1.9.6
PNPM_VERSION: 7.24.3
PNPM_VERSION: 7.32.2
NODE_MAINTENANCE_VERSION: 16
NODE_LTS_VERSION: 18
TEST_CONCURRENCY: 6
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pull_request_stats.yml
Expand Up @@ -7,7 +7,7 @@ name: Generate Pull Request Stats
env:
NAPI_CLI_VERSION: 2.14.7
TURBO_VERSION: 1.9.6
PNPM_VERSION: 7.24.3
PNPM_VERSION: 7.32.2
NODE_MAINTENANCE_VERSION: 16
NODE_LTS_VERSION: 18
TEST_CONCURRENCY: 6
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test_examples.yml
Expand Up @@ -14,7 +14,7 @@ on:
name: Test examples

env:
PNPM_VERSION: 7.24.3
PNPM_VERSION: 7.32.2

jobs:
testExamples:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/trigger_release.yml
Expand Up @@ -26,7 +26,7 @@ name: Trigger Release
env:
NAPI_CLI_VERSION: 2.14.7
TURBO_VERSION: 1.9.6
PNPM_VERSION: 7.24.3
PNPM_VERSION: 7.32.2
NODE_MAINTENANCE_VERSION: 16
NODE_LTS_VERSION: 18

Expand Down
2 changes: 1 addition & 1 deletion azure-pipelines.yml
Expand Up @@ -33,7 +33,7 @@ pr:

variables:
PNPM_CACHE_FOLDER: $(Pipeline.Workspace)/.pnpm-store
PNPM_VERSION: 7.24.3
PNPM_VERSION: 7.32.2
NEXT_TELEMETRY_DISABLED: '1'
node_16_version: ^16.8.0

Expand Down
4 changes: 2 additions & 2 deletions package.json
Expand Up @@ -51,7 +51,7 @@
"clean-trace-jaeger": "node scripts/rm.mjs test/integration/basic/.next && TRACE_TARGET=JAEGER pnpm next build test/integration/basic",
"debug": "cross-env NEXT_TELEMETRY_DISABLED=1 node --inspect packages/next/dist/bin/next",
"postinstall": "git config index.skipHash false && node scripts/install-native.mjs",
"version": "npx pnpm@7.24.3 install --no-frozen-lockfile && IS_PUBLISH=yes ./scripts/check-pre-compiled.sh && git add .",
"version": "npx pnpm@7.32.2 install --no-frozen-lockfile && IS_PUBLISH=yes ./scripts/check-pre-compiled.sh && git add .",
"prepare": "husky install",
"sync-react": "node ./scripts/sync-react.js",
"update-google-fonts": "node ./scripts/update-google-fonts.js"
Expand Down Expand Up @@ -239,5 +239,5 @@
"engines": {
"node": ">=16.8.0"
},
"packageManager": "pnpm@7.24.3"
"packageManager": "pnpm@7.32.2"
}

0 comments on commit 1602c2a

Please sign in to comment.