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

Add .nvmrc to fixtures/flight to automatically use Node 18 #26255

Closed
wants to merge 1 commit into from

Conversation

unstubbable
Copy link
Contributor

@unstubbable unstubbable commented Feb 27, 2023

The requirement for node 18 (global fetch) was introduced in f0cf832.

Summary

Because of the .nvmrc file in the project root, node 14 was automatically selected (with nvm being installed). When changing into fixtures/flight and running yarn dev, the dev server failed with ReferenceError: fetch is not defined (used in App.js).

How did you test this change?

cd fixtures/flight
yarn dev

Open browser at localhost:3000.

The requirement for Node 18 (global `fetch`) was introduced in f0cf832.
@unstubbable unstubbable changed the title Add .nvmrc to fixtures/flight to automatically use node 18 Add .nvmrc to fixtures/flight to automatically use Node 18 Feb 27, 2023
@react-sizebot
Copy link

Comparing: 6ff1733...06a215e

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 155.25 kB 155.25 kB = 48.98 kB 48.98 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 157.24 kB 157.24 kB = 49.64 kB 49.64 kB
facebook-www/ReactDOM-prod.classic.js = 532.63 kB 532.63 kB = 94.88 kB 94.88 kB
facebook-www/ReactDOM-prod.modern.js = 516.55 kB 516.55 kB = 92.47 kB 92.47 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 06a215e

@unstubbable
Copy link
Contributor Author

superseded by #26257

@unstubbable unstubbable deleted the flight-fixture-nvmrc branch February 28, 2023 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants