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

Added files for node version in reactive native project #34171

Closed
wants to merge 1 commit into from
Closed

Added files for node version in reactive native project #34171

wants to merge 1 commit into from

Conversation

ramonmedel
Copy link
Contributor

Summary:
Changelog:
[General][Added] - Added files for avn, nodenv, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

Summary:
Changelog:
[General][Added] - Added files for `avn`, `nodenv`, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

fbshipit-source-id: 5ea1eed66ca949b1074e3ab7d3599cad81993218
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner fb-exported labels Jul 11, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37683291

@react-native-bot react-native-bot added the Type: Enhancement A new feature or enhancement of an existing feature. label Jul 11, 2022
@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,821,538 +1,386
android hermes armeabi-v7a 7,214,628 +1,158
android hermes x86 8,135,385 +1,165
android hermes x86_64 8,113,163 +2,669
android jsc arm64-v8a 9,697,751 +0
android jsc armeabi-v7a 8,452,807 +0
android jsc x86 9,648,431 +0
android jsc x86_64 10,246,331 +0

Base commit: b834d58
Branch: main

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37683291

ramonmedel pushed a commit to ramonmedel/react-native that referenced this pull request Jul 11, 2022
Summary:
Pull Request resolved: facebook#34171

Changelog:
[General][Added] - Added files for `avn`, `nodenv`, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

fbshipit-source-id: 69ecee933d1664c67e5f48a9c5f4d1ad72ecac78
ramonmedel pushed a commit to ramonmedel/react-native that referenced this pull request Jul 11, 2022
Summary:
Pull Request resolved: facebook#34171

Changelog:
[General][Added] - Added files for `avn`, `nodenv`, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

fbshipit-source-id: 96795995bd5b1a6a33d57041c805da79162ba930
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37683291

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37683291

ramonmedel pushed a commit to ramonmedel/react-native that referenced this pull request Jul 11, 2022
Summary:
Pull Request resolved: facebook#34171

Changelog:
[General][Added] - Added files for `avn`, `nodenv`, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

fbshipit-source-id: 6f67755ad36ee785bcd336a43cf710b953f4e571
ramonmedel pushed a commit to ramonmedel/react-native that referenced this pull request Jul 11, 2022
Summary:
Pull Request resolved: facebook#34171

Changelog:
[General][Added] - Added files for `avn`, `nodenv`, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

fbshipit-source-id: 51fb67c15e357fdf103ad7aaea894d291161ab3a
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37683291

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D37683291

ramonmedel pushed a commit to ramonmedel/react-native that referenced this pull request Jul 11, 2022
Summary:
Pull Request resolved: facebook#34171

Changelog:
[General][Added] - Added files for `avn`, `nodenv`, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

fbshipit-source-id: 853c9d847e49b4921d0435434c4ce88cc3daa65d
facebook-github-bot pushed a commit that referenced this pull request Jul 11, 2022
Summary:
Pull Request resolved: #34171

Changelog:
[General][Added] - Added files for `avn`, `nodenv`, and other managers that set the node.js version in reactive native project including testing

Reviewed By: cortinico

Differential Revision: D37683291

fbshipit-source-id: ef0df0fb07f5cc0fa3591dde487f65e4b1b5f73f
@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @ramonmedel in 933fbb1.

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 Jul 11, 2022
@Adnan-Bacic
Copy link
Contributor

i guess if we are currently using a .nvmrc file from nvm, we dont need this file?

@ramonmedel
Copy link
Contributor Author

i guess if we are currently using a .nvmrc file from nvm, we dont need this file?

exactly

@KrisLau
Copy link

KrisLau commented Oct 12, 2022

How do we know if we need to add this file?

facebook-github-bot pushed a commit that referenced this pull request Nov 25, 2022
Summary:
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

Follow-up
- #34171
- #32980

Ref
- react-native-community/docker-android#187

cc ramonmedel cortinico gengjiawen

## 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
-->

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

Pull Request resolved: #35443

Test Plan: Everything builds and runs as expected

Reviewed By: cipolleschi

Differential Revision: D41531161

Pulled By: cortinico

fbshipit-source-id: 305888f55ed179f75bef34548aebf22fc2951308
hoxyq pushed a commit to hoxyq/react-native that referenced this pull request Jan 25, 2023
Summary:
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

Follow-up
- facebook#34171
- facebook#32980

Ref
- react-native-community/docker-android#187

cc ramonmedel cortinico gengjiawen

## 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
-->

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

Pull Request resolved: facebook#35443

Test Plan: Everything builds and runs as expected

Reviewed By: cipolleschi

Differential Revision: D41531161

Pulled By: cortinico

fbshipit-source-id: 305888f55ed179f75bef34548aebf22fc2951308
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
Summary:
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

Follow-up
- facebook#34171
- facebook#32980

Ref
- react-native-community/docker-android#187

cc ramonmedel cortinico gengjiawen

## 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
-->

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

Pull Request resolved: facebook#35443

Test Plan: Everything builds and runs as expected

Reviewed By: cipolleschi

Differential Revision: D41531161

Pulled By: cortinico

fbshipit-source-id: 305888f55ed179f75bef34548aebf22fc2951308
This pull request was closed.
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. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner Type: Enhancement A new feature or enhancement of an existing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants