Update jshint grunt task to exclude eqeqeq and curly for now, until we've resolved other code smells.

props kadamwhite.
see #25187.


git-svn-id: https://develop.svn.wordpress.org/trunk@26041 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Andrew Nacin 2013-11-07 21:14:49 +00:00
parent aacc286f7c
commit 8bdd289576
2 changed files with 41 additions and 46 deletions

View File

@ -3,20 +3,22 @@
"curly": true, "curly": true,
"eqeqeq": true, "eqeqeq": true,
"eqnull": true, "eqnull": true,
"es3": true,
"expr": true, "expr": true,
"immed": true, "immed": true,
"noarg": true, "noarg": true,
"onevar": true,
"quotmark": "single", "quotmark": "single",
"smarttabs": true, "smarttabs": true,
"trailing": true, "trailing": true,
"undef": true, "undef": true,
"unused": true, "unused": true,
"es3": true,
"onevar": true,
"browser": true, "browser": true,
"globals": { "globals": {
"_": false,
"Backbone": false,
"jQuery": false, "jQuery": false,
"wp": false "wp": false
} }

View File

@ -83,54 +83,47 @@ module.exports = function(grunt) {
jshint: { jshint: {
options: grunt.file.readJSON('.jshintrc'), options: grunt.file.readJSON('.jshintrc'),
grunt: { grunt: {
files: {
src: ['Gruntfile.js'] src: ['Gruntfile.js']
}, },
options: {
onevar: true
}
},
tests: { tests: {
files: {
src: [ src: [
'tests/qunit/**/*.js', 'tests/qunit/**/*.js',
'!tests/qunit/vendor/qunit.js' '!tests/qunit/vendor/qunit.js'
] ],
},
options: grunt.file.readJSON('tests/qunit/.jshintrc') options: grunt.file.readJSON('tests/qunit/.jshintrc')
}, },
'wp-admin': { core: {
files: { expand: true,
cwd: SOURCE_DIR,
src: [ src: [
'src/wp-admin/js/**/*.js', 'wp-admin/js/*.js',
'!src/wp-admin/js/farbtastic.js', 'wp-includes/js/*.js',
'!src/wp-admin/js/iris.min.js' // WordPress scripts inside directories
] 'wp-includes/js/jquery/jquery.table-hotkeys.js',
} 'wp-includes/js/mediaelement/wp-mediaelement.js',
}, 'wp-includes/js/plupload/handlers.js',
'wp-includes': { 'wp-includes/js/plupload/wp-plupload.js',
files: { 'wp-includes/js/tinymce/langs/wp-langs-en.js',
src: [ 'wp-includes/js/tinymce/plugins/wordpress/editor_plugin_src.js',
'src/wp-includes/js/**/*.js', 'wp-includes/js/tinymce/plugins/wp*/editor_plugin_src.js',
// 3rd-Party Scripts 'wp-includes/js/tinymce/mark_loaded_src.js',
'!src/wp-includes/js/backbone.min.js', 'wp-includes/js/tinymce/wp-tinymce-schema.js',
'!src/wp-includes/js/colorpicker.js', // Third party scripts
'!src/wp-includes/js/crop/**/*.js', '!wp-admin/js/farbtastic.js',
'!src/wp-includes/js/hoverIntent.js', '!wp-admin/js/iris.min.js',
'!src/wp-includes/js/imgareaselect/**/*.js', '!wp-includes/js/backbone.min.js',
'!src/wp-includes/js/jcrop/**/*.js', '!wp-includes/js/swfobject.js',
'!src/wp-includes/js/jquery/**/*.js', '!wp-includes/js/underscore.min.js',
'!src/wp-includes/js/json2.js', '!wp-includes/js/zxcvbn.min.js',
'!src/wp-includes/js/mediaelement/**/*.js', '!wp-includes/js/colorpicker.js',
'!src/wp-includes/js/plupload/**/*.js', '!wp-includes/js/hoverIntent.js',
'!src/wp-includes/js/swfobject.js', '!wp-includes/js/json2.js',
'!src/wp-includes/js/swfupload/**/*.js', '!wp-includes/js/tw-sack.js'
'!src/wp-includes/js/thickbox/**/*.js', ],
'!src/wp-includes/js/tinymce/**/*.js', // Remove once other JSHint errors are resolved
'!src/wp-includes/js/tw-sack.js', options: {
'!src/wp-includes/js/underscore.min.js', curly: false,
'!src/wp-includes/js/zxcvbn.min.js' eqeqeq: false
]
} }
} }
}, },