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

[build-utils] Add Node.js 16.x #7772

Merged
merged 2 commits into from
May 9, 2022
Merged

[build-utils] Add Node.js 16.x #7772

merged 2 commits into from
May 9, 2022

Conversation

styfle
Copy link
Member

@styfle styfle commented May 9, 2022

@styfle styfle added area: node semver: minor PR contains new features labels May 9, 2022
@styfle styfle requested a review from AndyBitz as a code owner May 9, 2022 13:10
@@ -4,6 +4,7 @@ import { NowBuildError } from '../errors';
import debug from '../debug';

const allOptions = [
{ major: 16, range: '16.x', runtime: 'nodejs16.x' },
Copy link

@gudmundur gudmundur May 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question: Does this mean that we'll default to NodeJS 16.x when this change is merged and deployed? If so, I wonder if want to gradually roll it out?

Copy link
Member Author

@styfle styfle May 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes if its a new project and there is no package.json engines.node defined.

Existing projects will not be changed.

@codecov
Copy link

codecov bot commented May 9, 2022

Codecov Report

Merging #7772 (914d8bf) into main (70b7db1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #7772   +/-   ##
=======================================
  Coverage   50.98%   50.98%           
=======================================
  Files         149      149           
  Lines        5762     5762           
  Branches     1455     1455           
=======================================
  Hits         2938     2938           
  Misses       2815     2815           
  Partials        9        9           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@styfle styfle requested a review from gudmundur May 9, 2022 13:22
@styfle styfle merged commit 3b9a987 into main May 9, 2022
@styfle styfle deleted the enable-nodejs16 branch May 9, 2022 14:13
JuanM04 added a commit to withastro/astro that referenced this pull request May 9, 2022
@pcjmfranken
Copy link

Released within 5 hours of AWS' announcement, nice work!

matthewp pushed a commit to withastro/astro that referenced this pull request May 11, 2022
* Removed ignores

* Migration to v3

* More changes

* Remove legacy redirects

* Fail when there is no ENABLE_VC_BUILD

* Fix edge

* Updated readme

* Changeset

* Added static mode

* Updated documentation

* Updated shim

* Made edge work!

* Updated changeset

* Ensure empty dir

* Fixed redirects for dynamic paths

* Removed extra declaration

* Splited imports

* Updated readme

* Fixed some urls

* Deprecated shim!

* [test]: Vercel NFT

* Beautify

* Edge bundle to node 14.19

Vercel runs 14.19.1 (I've checked it manually)

* Re-added shim (#3304)

* Added `node:` prefix

* Use the same bundling as Deno for Edge

* Remove esbuild

* Fixed shim

* Moved nft

* Updated changeset

* Added note about Edge

* fix typo

* Added support for Node 16 (vercel/vercel#7772)
natemoo-re pushed a commit to natemoo-re/astro-template-minimal that referenced this pull request Sep 30, 2022
* Removed ignores

* Migration to v3

* More changes

* Remove legacy redirects

* Fail when there is no ENABLE_VC_BUILD

* Fix edge

* Updated readme

* Changeset

* Added static mode

* Updated documentation

* Updated shim

* Made edge work!

* Updated changeset

* Ensure empty dir

* Fixed redirects for dynamic paths

* Removed extra declaration

* Splited imports

* Updated readme

* Fixed some urls

* Deprecated shim!

* [test]: Vercel NFT

* Beautify

* Edge bundle to node 14.19

Vercel runs 14.19.1 (I've checked it manually)

* Re-added shim (#3304)

* Added `node:` prefix

* Use the same bundling as Deno for Edge

* Remove esbuild

* Fixed shim

* Moved nft

* Updated changeset

* Added note about Edge

* fix typo

* Added support for Node 16 (vercel/vercel#7772)
mimed95 pushed a commit to mimed95/mimed95.github.io that referenced this pull request Apr 5, 2024
* Removed ignores

* Migration to v3

* More changes

* Remove legacy redirects

* Fail when there is no ENABLE_VC_BUILD

* Fix edge

* Updated readme

* Changeset

* Added static mode

* Updated documentation

* Updated shim

* Made edge work!

* Updated changeset

* Ensure empty dir

* Fixed redirects for dynamic paths

* Removed extra declaration

* Splited imports

* Updated readme

* Fixed some urls

* Deprecated shim!

* [test]: Vercel NFT

* Beautify

* Edge bundle to node 14.19

Vercel runs 14.19.1 (I've checked it manually)

* Re-added shim (#3304)

* Added `node:` prefix

* Use the same bundling as Deno for Edge

* Remove esbuild

* Fixed shim

* Moved nft

* Updated changeset

* Added note about Edge

* fix typo

* Added support for Node 16 (vercel/vercel#7772)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: node semver: minor PR contains new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants