feat(jshint): add jshint to build chain

jarkkopie · Jul 1, 2014 · decbf8f · decbf8f
1 parent 8be7f7c
commit decbf8f
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 0 deletions.
diff --git a/.jshintignore b/.jshintignore
@@ -0,0 +1 @@
+**/templates.js
diff --git a/gulpfile.js b/gulpfile.js
@@ -7,6 +7,8 @@ var concat = require('gulp-concat');
 var uglify = require('gulp-uglify');
 //var wrap = require('gulp-wrap');
 var templateCache = require('gulp-angular-templatecache');
+var gjshint = require('gulp-jshint');
+var stylish = require('jshint-stylish');
 
 var port = gutil.env.port || 3000;
 var covport = gutil.env.covport || 3001;
@@ -50,6 +52,8 @@ gulp.task('dist', ['tpl'], function () {
         'src/mentio.service.js',
         'dist/templates.js'
     ])
+    .pipe(gjshint())
+    .pipe(gjshint.reporter(stylish))
     .pipe(concat('mentio.js'))
 //    // hack to make componentjs libs to work
 //    .pipe(wrap('(function () {\n\n\'use strict\';\n\nvar Package = \'\';' +
@@ -60,6 +64,12 @@ gulp.task('dist', ['tpl'], function () {
     .pipe(gulp.dest('dist'));
 });
 
+gulp.task('jshint', function () {
+    return gulp.src('src/**/*.js')
+        .pipe(gjshint())
+        .pipe(gjshint.reporter(stylish));
+});
+
 gulp.task('tpl', function () {
     return gulp.src('src/**/*.tpl.html')
         .pipe(templateCache({

diff --git a/package.json b/package.json
@@ -27,12 +27,14 @@
     "gulp": "^3.6.2",
     "gulp-angular-templatecache": "^1.2.1",
     "gulp-concat": "^2.2.0",
+    "gulp-jshint": "^1.6.3",
     "gulp-karma": "0.0.4",
     "gulp-livereload": "^1.5.0",
     "gulp-uglify": "^0.3.1",
     "gulp-util": "^2.2.14",
     "gulp-wrap": "^0.3.0",
     "jasmine-reporters": "~0.4.1",
+    "jshint-stylish": "^0.2.0",
     "karma": "^0.12.16",
     "karma-chrome-launcher": "^0.1.2",
     "karma-coverage": "~0.1.4",