Skip to content

Conversation

@mzb
Copy link
Member

@mzb mzb commented Jul 13, 2017

Upgrade Node to v4.8.4 which contains some security fixes (ref:
https://nodejs.org/en/blog/vulnerability/july-2017-security-releases/).

I also upgraded node-sass-middleware bc of problems with building node-sass.

Upgrade Node to v4.8.4 which contains some security fixes (ref:
https://nodejs.org/en/blog/vulnerability/july-2017-security-releases/).
},
"engines": {
"node": "0.12.x"
"node": "4.8.4"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to add caret (^) in front of the version so the people using higher versions in major 4 could use it as well.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What @pdziok said.

Also, funky indentation yo.

@hnrc
Copy link
Member

hnrc commented Sep 21, 2017

👋 @mzb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants