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

chore: Update NodeJs Engine compat to v14, add pre-reqs to Composer readme #8225

Merged
merged 3 commits into from
Jul 1, 2021

Conversation

taicchoumsft
Copy link
Contributor

@taicchoumsft taicchoumsft commented Jun 30, 2021

Description

Update Node Version to V14. V16 is not supported yet by bf-cli (via orchestratorlib), and we've dropped support for Node V12 and below (via Antlr 4.9.2 in bf-lu)

Also update all our docs on this site to mention we only support building Composer on NodeV14 for now. @JonathanFingold has already done the accompanying update on the docs site here

Also add build pre-reqs to our README as a source of truth for docs.

Task Item

closes #8211

@cypress
Copy link

cypress bot commented Jun 30, 2021



Test summary

16 0 1 0Flakiness 0


Run details

Project Composer
Status Passed
Commit b0c198e
Started Jul 1, 2021 2:49 AM
Ended Jul 1, 2021 2:57 AM
Duration 08:10 💡
OS Linux Ubuntu - 20.04
Browser Electron 89

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@@ -27,7 +27,7 @@
"ejs": "^3.1.6"
},
"engines": {
"node": ">=12"
"node": "14.x"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@taicchoumsft Im assuming the build broke with Node 16, hence you are avoiding the usage of >=14?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is the error from bf-orchestrator when running yarn install on Node V16.4.0:

error @microsoft/bf-orchestrator@4.13.0-rc0: The engine "node" is incompatible with this module. Expected version "^10.0.0 || ^12.0.0 || ^13.0.0 || ^14.0.0". Got "16.4.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

I just created a ticket to track this on cli side: cli#1258


OS | Version | Architectures
------------------|--------------------|---------------
Mac OS X | 10.14+ | x64
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we note something about M1 (arm) version?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @hatpick , added a sentence explicitly mentioning we don't run on M1 hardware.

@coveralls
Copy link

coveralls commented Jul 1, 2021

Coverage Status

Coverage remained the same at 55.434% when pulling b0c198e on tachou/nodeV14 into 7d3433a on main.

@srinaath srinaath merged commit 9f061f9 into main Jul 1, 2021
@srinaath srinaath deleted the tachou/nodeV14 branch July 1, 2021 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build requires Node.js >=14 - but docs state otherwise
4 participants