Paladox has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/276043

Change subject: Add jshint to npm
......................................................................

Add jshint to npm

Change-Id: I9dd4f0e92bb944958476ac3e70879738abf45fbf
---
A .jshintignore
M .jshintrc
M Gruntfile.js
M package.json
4 files changed, 35 insertions(+), 2 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/EducationProgram 
refs/changes/43/276043/1

diff --git a/.jshintignore b/.jshintignore
new file mode 100644
index 0000000..3c3629e
--- /dev/null
+++ b/.jshintignore
@@ -0,0 +1 @@
+node_modules
diff --git a/.jshintrc b/.jshintrc
index 2c63c08..04c3a97 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -1,2 +1,23 @@
 {
+       // Enforcing
+       "bitwise": true,
+       "eqeqeq": true,
+       "freeze": true,
+       "latedef": "nofunc",
+       "noarg": true,
+       "nonew": true,
+       "undef": true,
+       "unused": true,
+       "strict": false,
+
+       // Relaxing
+       "es5": false,
+
+       // Environment
+       "browser": true,
+       "jquery": true,
+
+       "globals": {
+               "mediaWiki": false
+       }
 }
diff --git a/Gruntfile.js b/Gruntfile.js
index 9c56558..36b6671 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -1,9 +1,19 @@
 /*jshint node:true */
 module.exports = function ( grunt ) {
-       grunt.loadNpmTasks( 'grunt-banana-checker' );
+       grunt.loadNpmTasks( 'grunt-contrib-jshint' );
        grunt.loadNpmTasks( 'grunt-jsonlint' );
+       grunt.loadNpmTasks( 'grunt-banana-checker' );
 
        grunt.initConfig( {
+               jshint: {
+                       options: {
+                               jshintrc: true
+                       },
+                       all: [
+                               '**/*.js',
+                               '!node_modules/**'
+                       ]
+               },
                banana: {
                        all: 'i18n/'
                },
@@ -15,6 +25,6 @@
                }
        } );
 
-       grunt.registerTask( 'test', [ 'jsonlint', 'banana' ] );
+       grunt.registerTask( 'test', [ 'jshint', 'jsonlint', 'banana' ] );
        grunt.registerTask( 'default', 'test' );
 };
diff --git a/package.json b/package.json
index 72eb4aa..2b3860e 100644
--- a/package.json
+++ b/package.json
@@ -7,6 +7,7 @@
     "grunt": "0.4.5",
     "grunt-cli": "0.1.13",
     "grunt-banana-checker": "0.4.0",
+    "grunt-contrib-jshint": "1.0.0",
     "grunt-jsonlint": "1.0.7"
   }
 }

-- 
To view, visit https://gerrit.wikimedia.org/r/276043
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9dd4f0e92bb944958476ac3e70879738abf45fbf
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/EducationProgram
Gerrit-Branch: master
Gerrit-Owner: Paladox <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to