From d08c2e496d4d689b0188ddf0ab9c36e9f411b724 Mon Sep 17 00:00:00 2001 From: lilia Date: Wed, 7 Jan 2015 19:27:46 -1000 Subject: [PATCH] Re-enable sauce for PR's Revert "Do not run saucelabs tests on pull requests because they will fail" This reverts commit 7b851ba548ea8ab9774b59a6adcf5e27c9254360. 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. --- Gruntfile.js | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/Gruntfile.js b/Gruntfile.js index 8da414747..eadfef10c 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -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']);