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

{
	"extends": [
		"wikimedia/client",
		"wikimedia/jquery"
	],
	"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			`{`
build: Use eslint-config-wikimedia v0.9.0 and make pass Change-Id: Idb57fc12e7822cf17e10dbb726480fc7de0ae199 2018-11-21 12:33:32 +00:00			`"extends": [`
			`"wikimedia/client",`
			`"wikimedia/jquery"`
			`],`
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			`}`
			`}`