Skip to content

Conversation

@JPeer264
Copy link
Contributor

Node v7 should be included into the test suites. Many people are using v7 already.

@phated
Copy link
Member

phated commented Jan 11, 2017

People really shouldn't be using v7 in real environments. Also, we aren't able to test old gulp versions on v7 so I don't think we should include it. I was told not to worry about it until v8

@JPeer264
Copy link
Contributor Author

That's a good point. Ok for me it's new to not worry about it until v8, but I guess there are many good reasons

@phated
Copy link
Member

phated commented Jan 11, 2017

I've been told odd versions are experimental and even versions are the release. After an even version is cut, the odd version is no longer supported. We should probably remove the odd releases from our test matrix.

@JPeer264
Copy link
Contributor Author

That makes sense though. I am not sure about removing it. As long as they do not make noise and fail the tests it would be acceptable.

@JPeer264
Copy link
Contributor Author

@phated I removed now all uneven node versions from .travis.yml and appveyor.yml

@phated phated merged commit 3e648d7 into gulpjs:master Jan 13, 2017
@phated
Copy link
Member

phated commented Jan 13, 2017

Thanks @JPeer264

phated pushed a commit that referenced this pull request Dec 21, 2017
phated pushed a commit that referenced this pull request Dec 21, 2017
phated pushed a commit that referenced this pull request Dec 21, 2017
phated pushed a commit that referenced this pull request Dec 21, 2017
phated pushed a commit that referenced this pull request Dec 21, 2017
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