From f1488db109d13e748b071c02b40e90cdca1cc79d Mon Sep 17 00:00:00 2001 From: Lorenzo Sciandra Date: Fri, 28 Jan 2022 05:05:20 -0800 Subject: [PATCH] bump Node references from 12/14 to 14/16 (#32980) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Summary: Node 16 has been the LTS for quite some time now ([Oct 2021](https://nodejs.org/en/blog/release/v16.13.0/)), so this PR just wants to bring the RN OSS CI up to speed. (I realized that this was needed while doing the same for macos https://github.com/microsoft/react-native-macos/pull/997) ## Changelog [General] [Changed] - CI moved to Node 16. Pull Request resolved: https://github.com/facebook/react-native/pull/32980 Test Plan: CI itself is the test � locally no significant changes were experienced. Reviewed By: cortinico Differential Revision: D33821288 Pulled By: ShikaSD fbshipit-source-id: 4480ae1cb909e2b8d0b6abba4db76bbe71f0193e --- .circleci/config.yml | 12 ++++++------ package.json | 2 +- scripts/validate-ios-test-env.sh | 2 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index 120ad25cd5008a..a7929266e83033 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -28,11 +28,11 @@ executors: <<: *defaults docker: # Note: Version set separately for Windows builds, see below. - - image: circleci/node:14 + - image: circleci/node:16 nodeprevlts: <<: *defaults docker: - - image: circleci/node:12 + - image: circleci/node:14 reactnativeandroid: <<: *defaults docker: @@ -376,7 +376,7 @@ jobs: - run: name: Configure Environment Variables command: | - echo 'export PATH=/usr/local/opt/node@14/bin:$PATH' >> $BASH_ENV + echo 'export PATH=/usr/local/opt/node@16/bin:$PATH' >> $BASH_ENV source $BASH_ENV - with_brew_cache_span: @@ -393,7 +393,7 @@ jobs: name: Configure Node # Sourcing find-node.sh will ensure nvm is set up. # It also helps future invocation of find-node.sh prevent permission issue with nvm.sh. - command: source scripts/find-node.sh && nvm install 14 && nvm alias default 14 + command: source scripts/find-node.sh && nvm install 16 && nvm alias default 16 - run: name: Configure Watchman @@ -644,8 +644,8 @@ jobs: name: Install Node # Note: Version set separately for non-Windows builds, see above. command: | - nvm install 14.17.0 - nvm use 14.17.0 + nvm install 16 + nvm use 16 # Setup Dependencies - run: diff --git a/package.json b/package.json index 7fed48e771e3a7..179ad8a00f308c 100644 --- a/package.json +++ b/package.json @@ -7,7 +7,7 @@ "license": "MIT", "repository": "github:facebook/react-native", "engines": { - "node": ">=12" + "node": ">=14" }, "jest-junit": { "outputDirectory": "reports/junit", diff --git a/scripts/validate-ios-test-env.sh b/scripts/validate-ios-test-env.sh index 50d4efe9afc959..4cf714af8d5656 100755 --- a/scripts/validate-ios-test-env.sh +++ b/scripts/validate-ios-test-env.sh @@ -23,7 +23,7 @@ fi # Check that the correct version of node is installed NODE_VERSION="$(command node --version | sed 's/[-/a-zA-Z]//g' |sed 's/.\{2\}$//')" -if (( $(echo "${NODE_VERSION} < 12.0" | bc -l) )); then +if (( $(echo "${NODE_VERSION} < 14.0" | bc -l) )); then echo "Node ${NODE_VERSION} detected. This version of Node is not supported." echo "See https://reactnative.dev/docs/getting-started for instructions." exit 1