WMDE-leszek has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/354952 )
Change subject: Lint JavaScript with eslint instead of jscs and jshint
......................................................................
Lint JavaScript with eslint instead of jscs and jshint
Bug: T165843
Change-Id: I810cb04c101f5667aae515e5f2f9fe4a25dd6026
---
R .eslintignore
A .eslintrc.json
D .jscsrc
D .jshintrc
M Gruntfile.js
M package.json
6 files changed, 25 insertions(+), 66 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikibaseMediaInfo
refs/changes/52/354952/1
diff --git a/.jshintignore b/.eslintignore
similarity index 100%
rename from .jshintignore
rename to .eslintignore
diff --git a/.eslintrc.json b/.eslintrc.json
new file mode 100644
index 0000000..0246fe7
--- /dev/null
+++ b/.eslintrc.json
@@ -0,0 +1,17 @@
+{
+ "extends": "wikimedia",
+ "env": {
+ "browser": true,
+ "jquery": true
+ },
+ "globals": {
+ "util": false,
+ "wikibase": false
+ },
+ "rules": {
+ "indent": "off",
+ "no-underscore-dangle": "off",
+ "one-var": "off",
+ "vars-on-top": "off"
+ }
+}
diff --git a/.jscsrc b/.jscsrc
deleted file mode 100644
index b110fb8..0000000
--- a/.jscsrc
+++ /dev/null
@@ -1,30 +0,0 @@
-{
- "preset": "wikimedia",
- "excludeFiles": [ "node_modules/**", "vendor/**" ],
-
- // wikimedia overwrites
-
- "disallowDanglingUnderscores": null,
- "requireMultipleVarDecl": null,
-
- "jsDoc": {
- // what we don't yet follow is commented out
- "checkAnnotations": {
- "preset": "jsduck5",
- "extra": {
- "license": "some"
- }
- },
- "checkParamNames": true,
- "requireParamTypes": true,
- "checkRedundantParams": true,
- "checkReturnTypes": true,
- "checkRedundantReturns": true,
- "requireReturnTypes": true,
- "checkTypes": "strictNativeCase",
- "checkRedundantAccess": true,
- "requireNewlineAfterDescription": true
- },
-
- "requireVarDeclFirst": null
-}
diff --git a/.jshintrc b/.jshintrc
deleted file mode 100644
index 8de903d..0000000
--- a/.jshintrc
+++ /dev/null
@@ -1,22 +0,0 @@
-{
- // Enforcing
- "bitwise": true,
- "eqeqeq": true,
- "es3": true,
- "latedef": true,
- "noarg": true,
- "nonew": true,
- "undef": true,
- "unused": true,
- "strict": false,
-
- // Environment
- "browser": true,
-
- "globals": {
- "jQuery": false,
- "module": false,
- "util": false,
- "wikibase": false
- }
-}
diff --git a/Gruntfile.js b/Gruntfile.js
index ddcbbda..14793a1 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -1,22 +1,14 @@
-/*jshint node:true */
+/* eslint-env node */
+
module.exports = function ( grunt ) {
- grunt.loadNpmTasks( 'grunt-contrib-jshint' );
+ grunt.loadNpmTasks( 'grunt-eslint' );
grunt.loadNpmTasks( 'grunt-jsonlint' );
grunt.loadNpmTasks( 'grunt-banana-checker' );
- grunt.loadNpmTasks( 'grunt-jscs' );
grunt.loadNpmTasks( 'grunt-stylelint' );
grunt.initConfig( {
- jshint: {
- options: {
- jshintrc: true
- },
- all: [
- '**/*.js'
- ]
- },
- jscs: {
- src: '<%= jshint.all %>'
+ eslint: {
+ all: '.'
},
banana: {
all: 'i18n/'
@@ -38,6 +30,6 @@
}
} );
- grunt.registerTask( 'test', [ 'jshint', 'jscs', 'jsonlint', 'banana',
'stylelint' ] );
+ grunt.registerTask( 'test', [ 'eslint', 'jsonlint', 'banana',
'stylelint' ] );
grunt.registerTask( 'default', 'test' );
};
diff --git a/package.json b/package.json
index 07a0ec4..c608723 100644
--- a/package.json
+++ b/package.json
@@ -4,9 +4,11 @@
"test": "grunt test"
},
"devDependencies": {
+ "eslint-config-wikimedia": "0.4.0",
"grunt": "1.0.1",
"grunt-contrib-jshint": "0.11.3",
"grunt-banana-checker": "0.4.0",
+ "grunt-eslint": "19.0.0",
"grunt-jscs": "2.5.0",
"grunt-jsonlint": "1.0.7",
"grunt-stylelint": "0.6.0",
--
To view, visit https://gerrit.wikimedia.org/r/354952
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I810cb04c101f5667aae515e5f2f9fe4a25dd6026
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikibaseMediaInfo
Gerrit-Branch: master
Gerrit-Owner: WMDE-leszek <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
