Add supported versions of npm, Yarn and Node.js #628
Conversation
a6b2a89
to
2708c3d
Compare
package.json
Outdated
@@ -7,8 +7,9 @@ | |||
"bugs": "https://github.com/mozilla-neutrino/neutrino-dev/issues", | |||
"repository": "mozilla-neutrino/neutrino-dev", | |||
"engines": { | |||
"node": ">=6.10.0", | |||
"yarn": "^1.2.1" | |||
"node": ">=6.9.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.
The node
engine needs to be more complex to handle the actual support. I think you can use ||
to delineate these. Also, since 7
is excluded, we probably don't want >=
except in the 9
case.
"node": "^6.9.0 || ^8 || >=9"
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.
++
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.
I'm pretty sure the ^8
can be written as 8
(the latter being what I've seen used elsewhere).
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.
I haven't seen that, but I think sticking with the ^
makes the semver relationship clearer.
* Update packages in docs directory with new requirements * Update every monorepo package.json's engines * Update packages in packages directory with new requirements * Add missed requirements in docs directory * Modify node engine to be ^6.9.0 || ^8 || >=9
* Update packages in docs directory with new requirements * Update every monorepo package.json's engines * Update packages in packages directory with new requirements * Add missed requirements in docs directory * Modify node engine to be ^6.9.0 || ^8 || >=9
Fixes #625.