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
Match package.json node engines to the docs #35
Conversation
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.
Good cath! I’ve installed this several times on Node 10 and I never hit this. Have you got some config to force this behavior on npm or are you using yarn?
package.json
Outdated
@@ -4,7 +4,7 @@ | |||
"repository": "nearform/node-clinic", | |||
"version": "1.0.2", | |||
"engines": { | |||
"node": "^9.4.0 || ^8.9.4" | |||
"node": "^10.0.0 || ^8.9.4" |
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.
Thanks for the contribution can you keep the 9.4.0 part and just add the 10.0.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.
@mcollina updated.
Signed-off-by: Jonathan Cremin <jonathan@crem.in>
@mcollina I'm using yarn. I've actually just notice that there are some dependencies (at least |
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.
LGTM
Thanks!!!! |
It refuses to install against node 10.8.0 due to the spec in
package.json
. After adjusting node engines the tests are still passing.