Skip to content
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

Fix "yarn install" failure for contributors on Linux using Node 18 #8763

Merged
merged 1 commit into from
Jan 11, 2023

Conversation

alshdavid
Copy link
Contributor

↪️ Pull Request

When on Linux with Node 18, yarn install fails on msgpackr-extract. This PR upgrades the locked version of msgpackr-extract in the package lock to to one that is compatible.

💻 Examples

image

🚨 Test instructions

On Linux:

# Use node 18
nvm install 18
nvm use 18

# Install dependencies
yarn install

✔️ PR Todo

  • [] Added/updated unit tests for this change
  • Filled out test instructions (In case there aren't any unit tests)
  • [] Included links to related issues/PRs

@alshdavid alshdavid force-pushed the alsh/fix-msgpackr-extract-for-linux-on-node-18 branch from 0689990 to c7bfea4 Compare January 11, 2023 04:47
@parcel-benchmark
Copy link

parcel-benchmark commented Jan 11, 2023

Benchmark Results

Kitchen Sink ✅

Timings

Description Time Difference
Cold 1.45s -16.00ms
Cached 327.00ms -31.00ms 🚀

Cold Bundles

Bundle Size Difference Time Difference
dist/legacy/parcel.7cdb0fad.webp 102.94kb +0.00b 104.00ms +7.00ms ⚠️
dist/legacy/parcel.7cdb0fad.webp 102.94kb +0.00b 105.00ms +7.00ms ⚠️
dist/modern/parcel.7cdb0fad.webp 102.94kb +0.00b 105.00ms +8.00ms ⚠️

Cached Bundles

Bundle Size Difference Time Difference
dist/legacy/parcel.7cdb0fad.webp 102.94kb +0.00b 85.00ms -18.00ms 🚀
dist/legacy/parcel.7cdb0fad.webp 102.94kb +0.00b 84.00ms -19.00ms 🚀
dist/modern/parcel.7cdb0fad.webp 102.94kb +0.00b 85.00ms -18.00ms 🚀
dist/legacy/index.2c76ad23.js 1.66kb +0.00b 407.00ms -27.00ms 🚀
dist/legacy/index.8aaa89c9.js 1.20kb +0.00b 406.00ms -29.00ms 🚀
dist/modern/index.6be20f01.js 1.13kb +0.00b 406.00ms -28.00ms 🚀
dist/legacy/index.b8ae99ba.css 94.00b +0.00b 248.00ms -21.00ms 🚀
dist/modern/index.31cedca9.css 94.00b +0.00b 247.00ms -22.00ms 🚀

React HackerNews ✅

Timings

Description Time Difference
Cold 9.35s -43.00ms
Cached 421.00ms -14.00ms

Cold Bundles

No bundle changes detected.

Cached Bundles

No bundle changes detected.

AtlasKit Editor ✅

Timings

Description Time Difference
Cold 1.63m +267.00ms
Cached 2.13s -17.00ms

Cold Bundles

No bundle changes detected.

Cached Bundles

No bundle changes detected.

Three.js ✅

Timings

Description Time Difference
Cold 6.79s +57.00ms
Cached 253.00ms +1.00ms

Cold Bundles

No bundle changes detected.

Cached Bundles

No bundle changes detected.

Click here to view a detailed benchmark overview.

@mischnic mischnic merged commit 2172672 into v2 Jan 11, 2023
@mischnic mischnic deleted the alsh/fix-msgpackr-extract-for-linux-on-node-18 branch January 16, 2023 20:06
marcins pushed a commit to marcins/parcel that referenced this pull request Jul 14, 2023
* upstream/v2: (33 commits)
  v2.8.3
  Changelog for v2.8.3
  Address bug by updating an asset reference and merge conditions (parcel-bundler#8762)
  Fix CSS order when merging type change bundles (parcel-bundler#8766)
  fixing failing build for contributors on Linux using Node 18 (parcel-bundler#8763)
  Extension: Importers View and separate LSP protocol package (parcel-bundler#8747)
  Bump swc to fix sourcemaps with Windows line endings (parcel-bundler#8756)
  Apply HMR updates in topological order (parcel-bundler#8752)
  Make extension packaging work (parcel-bundler#8730)
  Typed api.storeResult (parcel-bundler#8732)
  Refactor LSP to use vscode-jsonrpc (parcel-bundler#8728)
  Bump swc (parcel-bundler#8742)
  Recursively check reachability when removing asset graphs from bundles in deduplication (parcel-bundler#6004)
  Fix tsc sourcemaps metadata (parcel-bundler#8734)
  Assigning to `this` in CommonJS (parcel-bundler#8737)
  Don't retarget dependencies if a symbol is imported multiple times with different local names (parcel-bundler#8738)
  Add a note about using flow in CONTRIBUTING.md (parcel-bundler#8731)
  filter out title execArgv to workers (parcel-bundler#8719)
  Document more of the BundleGraph class (parcel-bundler#8711)
  Fixed the hmr connection with host 0.0.0.0 (parcel-bundler#7357)
  ...
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.

3 participants