-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
v1.16.3 introduced "node": ">=10" depencency #160
Comments
Indeed, semver should be pinned at v6; this is a breaking change. |
Yep I overlooked this when using the sledgehammer of |
There’s no value in bumping semver or dropping support for older nodes; why rush to a v2? |
I think there’s value in keeping up with the latest versions of dependencies, if we assume later versions are generally better, faster and/or more secure. It also avoids your dependency tree from stagnating over time. There's no rush to v2, but it is happening soon (whenever I get time), so seems like a convenient time for ending support for Node 8. It reached end of life in December 2019 anyway, and probably means I can get rid of a bunch of the |
This is now done. |
In v1.16.3 packages were bumped, in particular
semver
from6.3.0
to7.1.3
. In this new version ofsemver
new dependency was introduced:https://github.com/npm/node-semver/blob/master/package.json#L36
I'm not sure if this could be considered as breaking change but productions servers with lower node version harmed.
The text was updated successfully, but these errors were encountered: