thegamer/Cable-Desktop
2
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Re-enable sauce for PR's

Browse source 
Revert "Do not run saucelabs tests on pull requests because they will fail"

This reverts commit 7b851ba548.

This seemed like a good idea at the time, but it prevents tests from
being run on PR's made from branches within our own fork. Also, in
hindsight, I find that the "merge with caution" alert banner is an
appropriate adornment for PR's which have not been able to run the
saucelabs tests due to their alien origin.
This commit is contained in:
lilia 2015-01-07 19:27:46 -10:00
parent 4e6faf4e27
commit d08c2e496d
1 changed files with 1 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
Gruntfile.js
View file

@ -149,13 +149,7 @@ module.exports = function(grunt) {
});
grunt.registerTask('dev', ['connect', 'watch']);
//Do not run saucelabs tests on pull requests because they will fail
var pullRequest = process.env.TRAVIS_PULL_REQUEST;
if (pullRequest ==="false") {
grunt.registerTask('test', ['jshint', 'connect', 'saucelabs-mocha']);
} else {
grunt.registerTask('test', ['jshint']);
}
grunt.registerTask('test', ['jshint', 'connect', 'saucelabs-mocha']);
grunt.registerTask('default', ['preen', 'concat', 'sass']);
grunt.registerTask('build', ['compile', 'concat:curve25519']);