-
-
Notifications
You must be signed in to change notification settings - Fork 7k
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
update Node to 6.x LTS #1228
update Node to 6.x LTS #1228
Conversation
Great call. |
Need to make sure everything works with the breaking changes from 4 -> 6. I'm pretty sure it'll be fine. |
Dunno why Travis' |
.nvmrc
Outdated
@@ -1 +1 @@ | |||
6.7.0 | |||
lts/boron |
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.
Perhaps it expects a number version? Travis version of nvm could be different from yours.. I have not really used nvm before either, so I am not sure.
.travis.yml
Outdated
@@ -28,7 +26,7 @@ before_install: | |||
- sudo apt-get -qq update | |||
- sudo apt-get -qq install g++-4.8 | |||
install: | |||
- nvm install $TRAVIS_NODE_VERSION | |||
- nvm install | |||
- npm install -g npm@3 |
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.
If Node 6+ comes with npm >=3, this line can be removed
ac94609
to
fd8dc07
Compare
The 4.x branch [entered maintenance](https://github.com/nodejs/LTS#lts-schedule1) on 1 April.
The Travis people replied; with |
The 4.x branch [entered maintenance](https://github.com/nodejs/LTS#lts-schedule1) on 1 April.
Merge upstream changes
…icles Make the "media gallery" not show up on Articles
The 4.x branch entered maintenance on 1 April.