Added lint task

Author: recipher

.eslintrc

@@ -0,0 +1,21 @@
+{
+  "parser": "babel-eslint"
+, "extends": "eslint:recommended"
+, "env": {
+    "browser": true
+  , "node": true
+  }
+, "rules": {
+    "strict": 0
+  }
+, "plugins": [
+    "react"
+  ]
+, "ecmaFeatures": {
+    "jsx": true
+  }
+, "rules": {
+    "comma-style": [ 2, "first" ]
+  , "no-unused-vars": [ 2, { "args": "after-used", "varsIgnorePattern": "React" }]
+  }
+}

lib/bump.js

@@ -1,6 +1,6 @@
 var bump = require('gulp-bump');
 
-module.exports = function(gulp, options) {
+module.exports = function(gulp) {
   var bumpVersion = function(type) {
     return gulp.src('./package.json')
     .pipe(bump({ type: type }))

lib/lint.js

@@ -0,0 +1,10 @@
+var eslint = require('gulp-eslint');
+
+module.exports = function(gulp) {
+  gulp.task('lint', function() {
+    return gulp.src([ 'lib/**/*.js', 'test/**/*.js' ])
+    .pipe(eslint())
+    .pipe(eslint.format())
+    .pipe(eslint.failAfterError());
+  });
+};

lib/publish.js

@@ -1,4 +1,4 @@
-module.exports = function(gulp, options) {
+module.exports = function(gulp) {
   gulp.task('publish', function(done) {
     require('child_process')
     .spawn('npm', [ 'publish' ], { stdio: 'inherit' })

lib/release.js

@@ -4,7 +4,7 @@ var release = function(type) {
   return sequence('test:release', 'tag:' + type, 'publish');
 };
 
-module.exports = function(gulp, options) {
+module.exports = function(gulp) {
   gulp.task('release', [ 'release:patch' ]);
 
   gulp.task('release:patch', release.bind(null, 'patch'));

lib/tag.js

@@ -1,7 +1,7 @@
 var path = require('path')
   , git = require('gulp-git');
 
-module.exports = function(gulp, options) {
+module.exports = function(gulp) {
   var getVersion = function() {
     var pkg = require(path.join(process.cwd(), 'package.json'));
     return pkg.version;
@@ -17,7 +17,7 @@ module.exports = function(gulp, options) {
     .on('end', done);
   };
 
-  var tag = function(type) {
+  var tag = function() {
     var version  = 'v' + getVersion()
       , message = 'Release ' + version;
 

lib/test.js

@@ -17,6 +17,7 @@ var noOp = function() {
 };
 
 var log = function(err) {
+  /* eslint no-console: 0 */
   console.log(err);
 };
 
@@ -47,19 +48,19 @@ module.exports = function(gulp, options) {
     .pipe(istanbul.hookRequire());
   });
 
-  gulp.task('test:unit', function() {
+  gulp.task('test:unit', [ 'lint' ], function() {
     return test('./test/unit/**/*.js');
   });
 
-  gulp.task('test:integration', function() {
+  gulp.task('test:integration', [ 'lint' ], function() {
     return test('./test/integration/**/*.js');
   });
 
-  gulp.task('test', [ 'test:coverage' ], function() {
+  gulp.task('test', [ 'lint', 'test:coverage' ], function() {
     return test('./test/**/*.js');
   });
 
-  gulp.task('test:release', [ 'test:coverage' ], function() {
+  gulp.task('test:release', [ 'lint', 'test:coverage' ], function() {
     return test('./test/**/*.js', { continue: true });
   });
 

package.json

@@ -14,9 +14,12 @@
     "url": "[email protected]:projectftbl/gulp.git"
   },
   "dependencies": {
+    "babel-eslint": "^4.1.6",
+    "eslint-plugin-react": "^3.14.0",
     "gulp": "^3.9.0",
     "gulp-bump": "^1.0.0",
     "gulp-env": "^0.2.0",
+    "gulp-eslint": "^1.1.1",
     "gulp-git": "^1.2.4",
     "gulp-istanbul": "^0.10.3",
     "gulp-mocha": "^2.1.3",