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
Add Node.js v8 to Travis CI #1743
Add Node.js v8 to Travis CI #1743
Conversation
I think we can drop 0.12 and 0.10. |
@abouthiroppy I think so too. |
Please follow up on this 🙇 |
@substack @maxogden What do you think? |
.travis.yml
Outdated
@@ -1,8 +1,6 @@ | |||
language: node_js | |||
node_js: | |||
- "8" | |||
- "7" |
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.
could you remove 7
and add 9
?
Note this will be a breaking change |
@stevemao I fix it. |
@ryysud sorry for the wait, curious if you could check out the test failure? it looks like it's a bug in the traceur version used by the |
i'll remove es6ify after merging. thanks for the patch! |
fwiw I think dropping node < 4 was a very very bad move, and I think that should be reverted. |
@ljharb Sorry. Thank you for your Pull Request 🙇 |
Add support for Node.js v8.
Updated .travis.yml.