[CI] Use LTS and LTS-1 Node versions #28048
Conversation
facebook-github-bot
added
CLA Signed
|
|
RNTester (Android/hermes/arm64-v8a): 3276800 bytes |
|
RNTester.app (iOS): 10657792 bytes |
There was a problem hiding this comment.
@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
|
This pull request was successfully merged by @hramos in 3611170. When will my fix make it into a release? | Upcoming Releases |
Summary: Previous config used Node10 and NodeLTS executors, but the naming wasn't accurate. As of today, Node 12 is LTS, Node 10 is also LTS but entering maintenance in April: https://nodejs.org/en/about/releases/ This PR switches all jobs to use Node 12 as nodelts, and renames test_js_lts to test_js_prevlts to reflect it's use of Node 10 (the previous Node LTS release). The executors are named NodeLTS and NodeLTSPrev. Later this year, we can switch these to use Node 14 and Node 12 as Node 10 goes into maintenance. ## Changelog [Internal] Pull Request resolved: facebook#28048 Test Plan: Circle CI Reviewed By: TheSavior Differential Revision: D19887240 Pulled By: hramos fbshipit-source-id: a8d553dca0f47a5ab6132a3880830a60a1b28736
Summary
Previous config used Node10 and NodeLTS executors, but the naming wasn't accurate. As of today, Node 12 is LTS, Node 10 is also LTS but entering maintenance in April: https://nodejs.org/en/about/releases/
This PR switches all jobs to use Node 12 as nodelts, and renames test_js_lts to test_js_prevlts to reflect it's use of Node 10 (the previous Node LTS release). The executors are named NodeLTS and NodeLTSPrev. Later this year, we can switch these to use Node 14 and Node 12 as Node 10 goes into maintenance.
Changelog
[Internal]
Test Plan
Circle CI