-
Notifications
You must be signed in to change notification settings - Fork 454
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(devs-infra): minimum node version at 10.21.0 #1793
Conversation
Pull Request Test Coverage Report for Build 5328
💛 - Coveralls |
@@ -130,6 +130,6 @@ | |||
] | |||
}, | |||
"engines": { | |||
"node": ">= 10" | |||
"node": ">= 10.21.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If a user has the engine-strict
flag set to true, they won't be able to use ts-jest with node versions < 10.21.0, right?
Do we want that behavior?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this change doesn't impact to end users unless we distribute ts-jest
with engineStrict: true
in our package.json
. I've tested the scenario that end users set engineStrict: true
but it doesn't prevent from installing ts-jest
. Only when we explicitly set in our package.json
, then end users won't be able to install.
So this change is safe, only for local ts-jest
development.
Sadly, this just broke my build on Google Cloud Functions.
I have no idea why it has this requirement, but GCF won't allow me to specify anything other than node 10
|
I think it's safer to revert this change, this will make sure all environments work. There is a configuration on GCF which cannot reproduce your problem with local machine. |
@lookfirst will be fixed in 26.1.3 , probably in a week or so because we want to see if any other things pop up. |
Thank you for the fast response. I didn't test heavily because it was late, but I had a very quick workaround which was to pin to |
I think |
Setting Even using FYI, Google Cloud Functions are currently using Node 10.18 If some earlier versions of node 10 have issues, it would be good to identify the truly incompatible ones and use that value instead of just choosing the latest version (10.21). Thanks for getting the change reverted. |
26.1.3 is out with the revert fix |
@ahnpnl thank you for your hard work on this and for the fast fix. |
Summary
Not all node 10 versions are supported by Jest, better we set minimum version node 10 at latest node 10 which is 10.21.0
This change only affects to development but not end users
Test plan
Green CI
Does this PR introduce a breaking change?
Other information
N.A.