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

Bump node version from 16 to 18 #35443

Closed
wants to merge 5 commits into from
Closed

Bump node version from 16 to 18 #35443

wants to merge 5 commits into from

Conversation

leotm
Copy link
Contributor

@leotm leotm commented Nov 23, 2022

Summary

Node 18: https://nodejs.org/en/blog/announcements/v18-release-announce/

Node 16 EOL: 2023-09-11
https://nodejs.org/en/blog/announcements/nodejs16-eol/

Node 18 EOL: 2025-04-30

Follow-up

Ref

cc @ramonmedel @cortinico @gengjiawen @kelset 🙌

Changelog

[JavaScript] [Changed] - Bump node version from 16 to 18

Test Plan

Everything builds and runs as expected


Follow-up

Update docs
https://reactnative.dev/contributing/changelogs-in-pull-requests

To reflect

/\[\s?(ANDROID|GENERAL|IOS|JS|JAVASCRIPT|INTERNAL)\s?\]\s?\[\s?(ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY)\s?\]\s*?-?\s*?(.*)/gi;

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 23, 2022
@analysis-bot
Copy link

analysis-bot commented Nov 23, 2022

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,102,569 +0
android hermes armeabi-v7a 6,470,637 +0
android hermes x86 7,520,037 +0
android hermes x86_64 7,378,724 +0
android jsc arm64-v8a 8,967,444 +0
android jsc armeabi-v7a 7,698,293 +0
android jsc x86 9,029,584 +0
android jsc x86_64 9,507,429 +0

Base commit: e11cfe9
Branch: main

Copy link
Contributor Author

@leotm leotm left a comment

Choose a reason for hiding this comment

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

likely follow-up w related non-deprecated changes: #32980

nodelts_image: &nodelts_image "cimg/node:16.14"

echo 'export PATH=/usr/local/opt/node@16/bin:$PATH' >> $BASH_ENV

@analysis-bot
Copy link

analysis-bot commented Nov 23, 2022

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: e11cfe9
Branch: main

@pull-bot
Copy link

PR build artifact for d4ecaee is ready.
To use, download tarball from "Artifacts" tab in this CircleCI job then run yarn add <path to tarball> in your React Native project.

@cortinico
Copy link
Contributor

I think we can do that, but you'll have to also update the Node version on CircleCI and make sure that everythign is green:
https://github.com/facebook/react-native/blob/main/.circleci/config.yml#L42-L43

Prev-LTS should be 16, and current should be 18.

Copy link
Contributor

@cortinico cortinico left a comment

Choose a reason for hiding this comment

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

(see my previous comment)

@leotm leotm changed the title Bump .node-version to 18 Bump node-version from 16 to 18 Nov 23, 2022
@leotm leotm changed the title Bump node-version from 16 to 18 Bump node version from 16 to 18 Nov 23, 2022
@pull-bot
Copy link

PR build artifact for e26ef04 is ready.
To use, download tarball from "Artifacts" tab in this CircleCI job then run yarn add <path to tarball> in your React Native project.

@leotm leotm requested a review from cortinico November 23, 2022 23:39
@kelset
Copy link
Contributor

kelset commented Nov 25, 2022

can you fix the changelog entry? [Javascript] is not a valid argument?

@leotm
Copy link
Contributor Author

leotm commented Nov 25, 2022

looks valid/appropriate no? so it's passing

/\[\s?(ANDROID|GENERAL|IOS|JS|JAVASCRIPT|INTERNAL)\s?\]\s?\[\s?(ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY)\s?\]\s*?-?\s*?(.*)/gi;

if js/javascript been deprecated, happy to create another PR to remove both opts and update to [General]

@facebook-github-bot
Copy link
Contributor

@cortinico has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@kelset
Copy link
Contributor

kelset commented Nov 25, 2022

looks valid/appropriate no? so it's passing

/\[\s?(ANDROID|GENERAL|IOS|JS|JAVASCRIPT|INTERNAL)\s?\]\s?\[\s?(ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY)\s?\]\s*?-?\s*?(.*)/gi;

if js/javascript been deprecated, happy to create another PR to remove both opts and update to [General]

oh wow then the docs are outdated 😅 https://reactnative.dev/contributing/changelogs-in-pull-requests <- that one will need to be fixed; thanks for spotting!

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @leotm in f75b92a.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Nov 25, 2022
facebook-github-bot pushed a commit that referenced this pull request Dec 5, 2022
…PR template (#35541)

Summary:
A few days back leotm pointed out how the dangerfile was disaligned with the docs around the valid changelog entries for categories and types here: #35443 (comment)

I dug a bit deeper into the whole thing and went to the source of truth, which is the changelog generator, in particular here: https://github.com/microsoft/rnx-kit/blob/main/incubator/rn-changelog-generator/src/utils/getChangeDimensions.ts

So I've updated the dangerfile to respect the options listed there. Also did a parallel PR on the website side: facebook/react-native-website#3447

So now all comms will be aligned.

## Changelog

<!-- Help reviewers and the release process by writing your own changelog entry. For an example, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests
-->

[INTERNAL] [FIXED] - fix changelog entries for dangerfiles and PR template

Pull Request resolved: #35541

Test Plan: N/A

Reviewed By: sammy-SC

Differential Revision: D41686913

Pulled By: cortinico

fbshipit-source-id: 2297b4a3209188bda4a609b60860de35c5e7e6df
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 20, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: c8db22bbbc8b3158cdf89e88114bee7aed188c12
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 20, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: 57399162a70030c56a2ead370e92a579b6105df2
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.
Node 16's EOL is 2023-09-11

https://nodejs.org/en/blog/announcements/nodejs16-eol/

- this follows up by formally making Node 18 the minimum supported version.

Docs PR:
facebook/react-native-website#3748

Changelog:
[General][Breaking] Bump minimum Node JS version to 18

Differential Revision: D46462639

fbshipit-source-id: 1f7d40f3753a88c1b284dc8cfc0a61084e9de8f5
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37791

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: cortinico, NickGerleman

Differential Revision: D46583997

fbshipit-source-id: 4041dd0a4b6bb94eeccc9b4a685d5fca06ab5175
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: 5007ca540bdcd478956d21cab405f805113b0264
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: 928e07deaeb32eeb8615d85174ae61be0de4b998
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.
Node 16's EOL is 2023-09-11

https://nodejs.org/en/blog/announcements/nodejs16-eol/

- this follows up by formally making Node 18 the minimum supported version.

Docs PR:
facebook/react-native-website#3748

Changelog:
[General][Breaking] Bump minimum Node JS version to 18

Differential Revision: D46462639

fbshipit-source-id: 372b07647d7578d9e8276f4426adf0583310c46d
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37791

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: cortinico, NickGerleman

Differential Revision: D46583997

fbshipit-source-id: 2d21d9283eca7109e9f4593bf60931f5ffc9ee56
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: b1f7bedde653cd47ffce55875ce5532db4e9ad63
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.
Node 16's EOL is 2023-09-11

https://nodejs.org/en/blog/announcements/nodejs16-eol/

- this follows up by formally making Node 18 the minimum supported version.

Docs PR:
facebook/react-native-website#3748

Changelog:
[General][Breaking] Bump minimum Node JS version to 18

Differential Revision: D46462639

fbshipit-source-id: cb9fcd600be5bc6cc47657b1f9dc87d01c7a2a2a
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37791

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: cortinico, NickGerleman

Differential Revision: D46583997

fbshipit-source-id: aa67de9670deb67dd2ab4c064cc074cade36a030
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: 8eb42fc009e55d6262b12e817bf8f7535054fbf3
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 21, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: 4d02982b86da00d3df8c627d41c5401444370663
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 22, 2023
Summary:
Pull Request resolved: facebook#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: 95d513197385bc993af49038aed11c4c082b5b77
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 22, 2023
Summary:
## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.
Node 16's EOL is 2023-09-11

https://nodejs.org/en/blog/announcements/nodejs16-eol/

- this follows up by formally making Node 18 the minimum supported version.

Docs PR:
facebook/react-native-website#3748

Changelog:
[General][Breaking] Bump minimum Node JS version to 18

Differential Revision: D46462639

fbshipit-source-id: 1b2ab0b9646a8aee0c66149deaa14d417eeb0b45
arushikesarwani94 added a commit to arushikesarwani94/react-native that referenced this pull request Jun 22, 2023
Summary:
Pull Request resolved: facebook#37791

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: cortinico, NickGerleman

Differential Revision: D46583997

fbshipit-source-id: b6fb7d884a885d1913a281e4ac5af830ab4bed4e
facebook-github-bot pushed a commit that referenced this pull request Jun 22, 2023
Summary:
Pull Request resolved: #37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In #35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: d3b607788f596e8b5a9c4a6855d3b2f4bafe9a7a
facebook-github-bot pushed a commit that referenced this pull request Jun 22, 2023
Summary:
Pull Request resolved: #37791

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In #35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: cortinico, NickGerleman

Differential Revision: D46583997

fbshipit-source-id: 1f31e2f205ac8b09494c2a7d3b73b9f36eff221b
@cipolleschi cipolleschi mentioned this pull request Oct 11, 2023
blakef pushed a commit to blakef/template that referenced this pull request Feb 28, 2024
Summary:
Pull Request resolved: facebook/react-native#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook/react-native#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: d3b607788f596e8b5a9c4a6855d3b2f4bafe9a7a

Original: facebook/react-native@43a6ea9
blakef pushed a commit to react-native-community/template that referenced this pull request Feb 29, 2024
Summary:
Pull Request resolved: facebook/react-native#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook/react-native#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: d3b607788f596e8b5a9c4a6855d3b2f4bafe9a7a

Original-Commit: facebook/react-native@43a6ea9
blakef pushed a commit to react-native-community/template that referenced this pull request Feb 29, 2024
Summary:
Pull Request resolved: facebook/react-native#37709

## Bump minimum Node JS version to 18 via `react-native/package.json#engines`

In facebook/react-native#35443 we bumped up the node version from 16 to 18.

Node 16 [ends maintenance releases on 2023-09-11](https://nodejs.org/en/blog/announcements/nodejs16-eol/), and bumping this minimum will allow other associated Node JS tools (CLI, Metro, Jest) to reduce their support burden.

This follows up by formally making Node 18 the minimum supported version.

**Docs PR:**
facebook/react-native-website#3748

**Changelog:**
[General][Breaking] Bump minimum Node JS version to 18

Reviewed By: yungsters

Differential Revision: D46462639

fbshipit-source-id: d3b607788f596e8b5a9c4a6855d3b2f4bafe9a7a

Original-Commit: facebook/react-native@43a6ea9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants