chore(devs-infra): minimum node version at 10.21.0#1793
Conversation
Pull Request Test Coverage Report for Build 5328
💛 - Coveralls |
| }, | ||
| "engines": { | ||
| "node": ">= 10" | ||
| "node": ">= 10.21.0" |
There was a problem hiding this comment.
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.
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.