Merge pull request #79 from FullstackAcademy/eslint-env

Split production and dev linting

Author: joedotjs

generated/.eslintrc.json

@@ -12,7 +12,8 @@
   },
   "rules": {
     "global-require": 0, // 0 = off, 1 = warn (yellow), 2 = error (red)
-    "camelcase": 0
+    "camelcase": 0,
+    "no-debugger": 1
     // add more rules here to override the fullstack config
   }
 }

generated/gulpfile.js

@@ -109,6 +109,22 @@ gulp.task('buildCSS', function () {
 // Production tasks
 // --------------------------------------------------------------
 
+gulp.task('lintJSProduction', function () {
+
+    return gulp.src(['./browser/js/**/*.js', './server/**/*.js'])
+        .pipe(plumber({
+            errorHandler: notify.onError('Linting FAILED! Check your gulp process.')
+        }))
+        .pipe(eslint({
+            rules: {
+                'no-debugger': 2 // 1 in dev
+            }
+        }))
+        .pipe(eslint.format())
+        .pipe(eslint.failOnError());
+
+});
+
 gulp.task('buildCSSProduction', function () {
     return gulp.src('./browser/scss/main.scss')
         .pipe(sass())