mediawiki-extensions-Visual.../.eslintrc.json

{
	"extends": "wikimedia",
	"env": {
		"browser": true,
		"jquery": true
	},
	"globals": {
		"ve": true,
		"VisualEditorSupportCheck": false,
		"OO": false,
		"unicodeJS": false,
		"RangeFix": false,
		"Papa": false,
		"mw": false,
		"require": false
	},
	"rules": {
		"dot-notation": [ "error", { "allowKeywords": true } ],
		"max-statements-per-line": [ "error", { "max": 1 } ],
		"max-len": 0,
		"valid-jsdoc": 0
	}
}
build: Replace jscs and jshint with eslint It's new, it's fresh, it's amazing, it's here. Change-Id: I5dc784411f704685ed5cc763a2b2b1c5d3e5a610 2016-10-28 00:22:30 +00:00			`{`
			`"extends": "wikimedia",`
			`"env": {`
			`"browser": true,`
eslint: Move qunit env flag to test config Change-Id: I290c436b25e306a3657324304ded08011ffcea7a 2018-06-05 20:12:56 +00:00			`"jquery": true`
build: Replace jscs and jshint with eslint It's new, it's fresh, it's amazing, it's here. Change-Id: I5dc784411f704685ed5cc763a2b2b1c5d3e5a610 2016-10-28 00:22:30 +00:00			`},`
			`"globals": {`
			`"ve": true,`
			`"VisualEditorSupportCheck": false,`
			`"OO": false,`
			`"unicodeJS": false,`
			`"RangeFix": false,`
			`"Papa": false,`
mw.DesktopArticleTarget: Use require() for mw.page.watch Add 'require' to eslint global whitelist for this extension, since ResourceLoader provides this in all contexts now. Bug: T166254 Depends-On: I0e6a95bd4fbaba06eae5137ac7af84f62bebf6f0 Change-Id: I6b2fe57317dcff18ec6d9ea0cd9e4eb9c894c169 2017-05-23 19:11:47 +00:00			`"mw": false,`
			`"require": false`
build: Replace jscs and jshint with eslint It's new, it's fresh, it's amazing, it's here. Change-Id: I5dc784411f704685ed5cc763a2b2b1c5d3e5a610 2016-10-28 00:22:30 +00:00			`},`
			`"rules": {`
eslint: Re-enable wrap-iife and partially enable dot-notation Change-Id: I4420d66f528dcb18ebe2b9f63996661b2969f833 2016-11-12 14:43:43 +00:00			`"dot-notation": [ "error", { "allowKeywords": true } ],`
.eslintrc: Enable and fix "max-statements-per-line": 1 I am surprised this was disabled. I investigated this after reviewing some code by a new contributor which I was certain should have failed the lint check, but passed. Change-Id: I5b3c837b8ca3292f6e268b3922443bd9587eadbe 2018-10-29 23:19:16 +00:00			`"max-statements-per-line": [ "error", { "max": 1 } ],`
build: Bump linters for eslint 5 and make pass Change-Id: I88627028e5806c2fdceb76fd26bcb57d44ed4c20 2018-07-05 23:59:22 +00:00			`"max-len": 0,`
eslint: Remove duplicate rule (no-alert) Change-Id: I040689ddf8fa67c89f8b7bf154be8c1ceff2e5b8 2017-04-29 11:21:40 +00:00			`"valid-jsdoc": 0`
build: Replace jscs and jshint with eslint It's new, it's fresh, it's amazing, it's here. Change-Id: I5dc784411f704685ed5cc763a2b2b1c5d3e5a610 2016-10-28 00:22:30 +00:00			`}`
			`}`