-
Notifications
You must be signed in to change notification settings - Fork 71.3k
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 0.12 to travis #415
Conversation
@@ -1,14 +1,12 @@ | |||
language: node_js | |||
sudo: false |
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.
A tweak to allow Travis to start up faster. More info here.
@@ -29,7 +29,7 @@ | |||
"postinstall": "node node_modules/bower/bin/bower install" | |||
}, | |||
"engines": { | |||
"node": "~0.10.0" | |||
"node": "0.10.x" |
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.
Equivalent, just switching to the prevailing style. I assume we want to do more thorough testing before adding or switching to "0.12.x"
.
|
||
travis: test travis-cov |
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.
Redundant – it's not necessary to separate testing and generating the coverage report.
allow_failures: | ||
- node_js: "0.11" | ||
- "0.10" | ||
- "0.12" | ||
services: | ||
- mongodb | ||
before_script: | ||
- sleep 10 | ||
- echo mongo mongo_travis |
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.
Just curious – what is this supposed to do?
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.
Maybe left over from testing the Travis setup? @bewest?
7adef4f
to
650074b
Compare
Need a new PR that targets dev too, since we're only updating master with releases |
Abandoning to target dev branch |
Resolves #414. Also includes some travis-related cleanup, cutting the Travis build time in half (to <1 minute under normal load).