Skip to content
This repository has been archived by the owner on Feb 23, 2019. It is now read-only.

.travis.yml: Test on Node 10, but not 9 #232

Closed
wants to merge 2 commits into from

Conversation

rhoggSugarcrm
Copy link
Contributor

Node.js 9 isn't an LTS release, but 10 will be (eventually).

Signed-off-by: Bob Wombat Hogg rhogg@sugarcrm.com

@rhoggSugarcrm rhoggSugarcrm self-assigned this Apr 26, 2018
@coveralls
Copy link

coveralls commented Apr 26, 2018

Coverage Status

Coverage remained the same at 95.977% when pulling ff231fd on rhoggSugarcrm:test-on-node-10 into 206e018 on sugarcrm:master.

@rhoggSugarcrm
Copy link
Contributor Author

rhoggSugarcrm commented Apr 26, 2018

Looks like there's a problem with Gulp on Node 10.

[EDIT: Both gulp and gulp-cli's test suites pass on Node 10, so it looks like the problem is either in our own code or in one of our dependencies.]

Signed-off-by: Bob Wombat Hogg <rhogg@sugarcrm.com>
Signed-off-by: Bob Wombat Hogg <rhogg@sugarcrm.com>
@rhoggSugarcrm
Copy link
Contributor Author

Looks like upgrading to gulp 4.x fixes it

@rhoggSugarcrm rhoggSugarcrm deleted the test-on-node-10 branch February 7, 2019 19:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants