[Build] Move JSHint config to .jshintrc

...to allow code editors etc to pick up on rules, per
https://github.com/nasa/openmct/pull/724#issuecomment-193542314
This commit is contained in:
Victor Woeltjen 2016-04-08 16:37:18 -07:00
parent c1ae68b565
commit a224711dce
2 changed files with 24 additions and 24 deletions

22
.jshintrc Normal file
View File

@ -0,0 +1,22 @@
{
"bitwise": true,
"browser": true,
"curly": true,
"eqeqeq": true,
"forin": true,
"freeze": true,
"funcscope": true,
"futurehostile": true,
"latedef": true,
"noarg": true,
"nocomma": true,
"nonbsp": true,
"nonew": true,
"predef": [
"define",
"Promise"
],
"strict": "implied",
"undef": true,
"unused": "vars"
}

View File

@ -58,28 +58,6 @@ var gulp = require('gulp'),
mainConfigFile: paths.main, mainConfigFile: paths.main,
wrapShim: true wrapShim: true
}, },
jshint: {
"bitwise": true,
"browser": true,
"curly": true,
"eqeqeq": true,
"forin": true,
"freeze": true,
"funcscope": true,
"futurehostile": true,
"latedef": true,
"noarg": true,
"nocomma": true,
"nonbsp": true,
"nonew": true,
"predef": [
"define",
"Promise"
],
"strict": "implied",
"undef": true,
"unused": "vars"
},
karma: { karma: {
configFile: path.resolve(__dirname, 'karma.conf.js'), configFile: path.resolve(__dirname, 'karma.conf.js'),
singleRun: true singleRun: true
@ -128,9 +106,9 @@ gulp.task('lint', function () {
return "!" + glob; return "!" + glob;
}), }),
scriptLint = gulp.src(paths.scripts.concat(nonspecs)) scriptLint = gulp.src(paths.scripts.concat(nonspecs))
.pipe(jshint(options.jshint)), .pipe(jshint()),
specLint = gulp.src(paths.specs) specLint = gulp.src(paths.specs)
.pipe(jshint(_.extend({ jasmine: true }, options.jshint))); .pipe(jshint({ jasmine: true }));
return merge(scriptLint, specLint) return merge(scriptLint, specLint)
.pipe(jshint.reporter('default')) .pipe(jshint.reporter('default'))