-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Conversation
Node 18: https://nodejs.org/de/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 Ref - react-native-community/docker-android#187 cc @ramonmedel @cortinico @gengjiawen
Base commit: e11cfe9 |
There was a problem hiding this 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
react-native/.circleci/config.yml
Line 42 in c5a8425
nodelts_image: &nodelts_image "cimg/node:16.14" |
react-native/.circleci/config.yml
Line 565 in c5a8425
echo 'export PATH=/usr/local/opt/node@16/bin:$PATH' >> $BASH_ENV |
Base commit: e11cfe9 |
PR build artifact for d4ecaee is ready. |
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: Prev-LTS should be 16, and current should be 18. |
There was a problem hiding this 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)
PR build artifact for e26ef04 is ready. |
can you fix the changelog entry? [Javascript] is not a valid argument? |
looks valid/appropriate no? so it's passing
if js/javascript been deprecated, happy to create another PR to remove both opts and update to [General] |
@cortinico has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
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! |
This pull request was successfully merged by @leotm in f75b92a. When will my fix make it into a release? | Upcoming Releases |
…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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
react-native/packages/react-native-bots/dangerfile.js
Line 62 in 55b09da