Permalink
Browse files

jshint to replace jslint

  • Loading branch information...
1 parent ae59ad4 commit 54baddd876675974a0d6b8be23cd517c36b3c3f7 @dandehavilland dandehavilland committed Jan 22, 2015
Showing with 11 additions and 10 deletions.
  1. +2 −0 .jshintignore
  2. +2 −1 .jshintrc
  3. +6 −8 gulp/index.js
  4. +1 −1 package.json
View
@@ -0,0 +1,2 @@
+src/js/common/bootstrap_tooltip_popover.js
+src/js/layout/bootstrap_dropdown.js
View
@@ -5,5 +5,6 @@
"MG": false,
"d3": false
},
- "laxbreak": true
+ "laxbreak": true,
+ "validthis": true
}
View
@@ -8,7 +8,7 @@ var
rename = require('gulp-rename'),
//sass = require('gulp-sass'), // for building css from scss
//minifycss = require('gulp-minify-css'), // for minifiing css
- jslint = require('gulp-jslint'),
+ jshint = require('gulp-jshint'),
testem = require('gulp-testem'),
connect = require('gulp-connect'),
es6ModuleTranspiler = require("gulp-es6-module-transpiler");
@@ -45,7 +45,7 @@ var
];
-gulp.task('default', ['jslint', 'test', 'build:js']);
+gulp.task('default', ['jshint', 'test', 'build:js']);
gulp.task('clean', function () {
return gulp.src([dist + 'metricsgraphics.js', dist + 'metricsgraphics.min.js'], {read: false})
@@ -100,13 +100,11 @@ gulp.task('build:js', ['clean'], function () {
.pipe(gulp.dest(dist));
});
-// Check source js files with jslint
-gulp.task('jslint', function () {
+// Check source js files with jshint
+gulp.task('jshint', function () {
return gulp.src(jsFiles)
- .pipe(jslint({
- predef: ["window", '$', 'd3'], // used globals
- nomen: false // true if there are variable names with leading _
- }));
+ .pipe(jshint())
+ .pipe(jshint.reporter('default'));
});
// Run test suite server (testem')
View
@@ -41,7 +41,7 @@
"gulp-concat": "^2.4.2",
"gulp-connect": "^2.2.0",
"gulp-es6-module-transpiler": "^0.2.0",
- "gulp-jslint": "^0.2.1",
+ "gulp-jshint": "^1.9.0",
"gulp-rename": "^1.2.0",
"gulp-rimraf": "^0.1.1",
"gulp-testem": "0.0.1",

0 comments on commit 54baddd

Please sign in to comment.