Skip to content
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

Use node 6 on travis #130

Merged
merged 1 commit into from
Nov 30, 2016
Merged

Use node 6 on travis #130

merged 1 commit into from
Nov 30, 2016

Conversation

nikolas
Copy link
Contributor

@nikolas nikolas commented Nov 30, 2016

Version 6 is the current LTS branch: https://github.com/nodejs/LTS#lts-schedule

@cookpete
Copy link
Owner

Build is only failing due to an unrelated lint error, so I'll merge this up and fix the lint issue on master. Thanks!

@cookpete cookpete merged commit 5a5dc7d into cookpete:master Nov 30, 2016
@cookpete
Copy link
Owner

Probably should just change this to node, to always use the latest.

https://docs.travis-ci.com/user/languages/javascript-with-nodejs#Available-Versions

@nikolas
Copy link
Contributor Author

nikolas commented Dec 1, 2016

Oh interesting. Yeah it's up to you. Personally, I like having to consciously update the version. Then it's more obvious what the problem is if any issues come up. But of course it's your project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants