Merge pull request #27 from sirap-group/26-break-on-lint-errors-for-t…

…ravis-build Fix #26 break on lint errors for travis build
sirap-group · Jan 4, 2017 · f81d85d · f81d85d
2 parents 94874e2 + 06b634a
commit f81d85d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -6,5 +6,4 @@ before_script:
   - npm install -g gulp-cli
   - npm install -g jscover
 script:
-  - npm test
-  - npm run coverage
+  - npm run lint && npm run coverage
diff --git a/gulpfile.js b/gulpfile.js
@@ -20,7 +20,7 @@ gulp.task('lint', function () {
   return gulp.src(['./gulpfile.js', './lib/**/*.js', './tests/**/*.spec.js'])
   .pipe(linter())
   .pipe(linter.reporter('default', {
-    breakOnError: false
+    breakOnError: argv.breakOnError
   }))
 })
 

diff --git a/package.json b/package.json
@@ -6,6 +6,7 @@
   "scripts": {
     "test": "gulp test",
     "TDD": "gulp test 2> /dev/null; gulp watch",
+    "lint": "gulp lint --breakOnError",
     "coverage": "gulp coverage"
   },
   "repository": {