Skip to content

Commit

Permalink
Use Node v14 in Windows CircleCI jobs (#31656)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #31656

CircleCI's Windows executor currently ships with a pre-LTS release of Node v12, breaking our Windows jobs ([example](https://app.circleci.com/pipelines/github/facebook/react-native/9280/workflows/21e6e59c-d853-47a1-af62-1368c8ce10ce/jobs/203983)) following #30637, ultimately due to jestjs/jest#10685 dropping support for non-LTS versions in the Node v12 release line.

Luckily, the Windows executor [does ship with nvm](circleci/circleci-docs#3733) so we can use that to install a desired Node version. Rather than just pinning a later v12 release that is LTS, we align the Windows config with the one for non-Windows jobs (in the `nodelts` executor) by running against the latest available Node v14 release.

Changelog:
[Internal]

Differential Revision: D28896581

fbshipit-source-id: cdcf9341e265e3bf28457d3e91c26d76a1dbb82a
  • Loading branch information
motiz88 authored and facebook-github-bot committed Jun 4, 2021
1 parent b637b21 commit 2eb5d14
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions .circleci/config.yml
Expand Up @@ -27,6 +27,7 @@ executors:
nodelts:
<<: *defaults
docker:
# Note: Version set separately for Windows builds, see below.
- image: circleci/node:14
nodeprevlts:
<<: *defaults
Expand Down Expand Up @@ -624,6 +625,13 @@ jobs:
steps:
- checkout

- run:
name: Install Node
# Note: Version set separately for non-Windows builds, see above.
command: |
nvm install 14
nvm use 14
# Setup Dependencies
- run:
name: Install Yarn
Expand Down

0 comments on commit 2eb5d14

Please sign in to comment.