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

Travis: Add Node v4 #195

Merged
merged 1 commit into from
Sep 28, 2015
Merged

Travis: Add Node v4 #195

merged 1 commit into from
Sep 28, 2015

Conversation

wbinnssmith
Copy link
Contributor

I imagine the entry "4" will continue to be the latest stable release of the 4.x line on travis.

@wbinnssmith wbinnssmith changed the title Add Node v4 and remove iojs Travis: Add Node v4 and remove iojs Sep 28, 2015
@ljharb
Copy link
Collaborator

ljharb commented Sep 28, 2015

I don't see any reason to remove iojs if it continues to pass. Also, instead of 4, you can use node to test the latest version of node.

@wbinnssmith
Copy link
Contributor Author

Hm. When node hits 5 (which it will in the coming months iirc), "node" will simply move onto 5 (unless travis decides to keep node pointing to the latest LTS instead of the latest node) and 4 will become untested. Either way, I imagine it will be best to be explicit. See browserify's patch.

@ljharb
Copy link
Collaborator

ljharb commented Sep 28, 2015

¯_(ツ)_/¯ not sure what node LTS has to do with an arbitrary library's chosen support level.

I don't mind putting 4 in here - not 4.0.0, though - but I still think it's useful to keep iojs.

@wavded
Copy link

wavded commented Sep 28, 2015

I second that its still useful to keep iojs.

I imagine the entry "4" will continue to be the latest stable release of the 4.x line on travis.
@wbinnssmith wbinnssmith changed the title Travis: Add Node v4 and remove iojs Travis: Add Node v4 Sep 28, 2015
@wbinnssmith
Copy link
Contributor Author

All right, the patch simply adds "4" now.

@ljharb
Copy link
Collaborator

ljharb commented Sep 28, 2015

LGTM

ljharb added a commit that referenced this pull request Sep 28, 2015
@ljharb ljharb merged commit 1e22819 into tape-testing:master Sep 28, 2015
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.

None yet

3 participants