Skip to content

Commit

Permalink
Build: remove node .10 from travis
Browse files Browse the repository at this point in the history
  • Loading branch information
markelog committed Dec 22, 2015
1 parent 14eba98 commit 498fd24
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion .travis.yml
@@ -1,7 +1,6 @@
language: node_js
sudo: false
node_js:
- "0.10"
- "0.12"
- "4"
- "5"

5 comments on commit 498fd24

@mgol
Copy link
Member

@mgol mgol commented on 498fd24 Apr 26, 2016

Choose a reason for hiding this comment

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

@markelog I've only noticed it now; why the removal? Node 0.10 is still supported by upstream; also, the master & 2.2-stable branches are still tested on Node 0.10. I think 1.12-stable should be tested there as well, at least until upstream drops support for this version.

@mgol
Copy link
Member

@mgol mgol commented on 498fd24 Apr 26, 2016

Choose a reason for hiding this comment

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

If the reason was instability of npm@1 then Node 0.10.44 updated npm to 2.15.1 so that should no longer be an issue.

@markelog
Copy link
Member Author

@markelog markelog commented on 498fd24 Apr 26, 2016

Choose a reason for hiding this comment

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

That might be it, but i honestly don't remember, i'm all for brining that back

@mgol
Copy link
Member

@mgol mgol commented on 498fd24 Apr 26, 2016

Choose a reason for hiding this comment

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

OK, thanks! I didn't want to do anything behind your back. I'll re-add it then.

@markelog
Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you for the consideration, i appreciate it :)

Please sign in to comment.