Build: Remove obsolete globals from ESLint configuration

mgol · web-flow · commit c10945d0e15c · 2019-02-19T13:20:57.000+01:00
We had quite a few obsolete globals declared in various ESLint config files. We also no longer allow to rely on the `noGlobal` & `jQuery` globals in the built file which is not needed. Closes gh-4301
diff --git a/.eslintrc-browser.json b/.eslintrc-browser.json
@@ -19,10 +19,8 @@
 
 	"globals": {
 		"window": true,
-		"jQuery": true,
 		"define": true,
-		"module": true,
-		"noGlobal": true
+		"module": true
 	},
 
 	"rules": {
diff --git a/src/.eslintrc.json b/src/.eslintrc.json
@@ -1,5 +1,9 @@
 {
 	"root": true,
 
-	"extends": "../.eslintrc-browser.json"
+	"extends": "../.eslintrc-browser.json",
+
+	"globals": {
+		"jQuery": true
+	}
 }
diff --git a/test/.eslintrc.json b/test/.eslintrc.json
@@ -12,22 +12,15 @@
 
 	"globals": {
 		"require": false,
-		"define": false,
-		"DOMParser": false,
-		"JSON": false,
 		"Promise": false,
 		"Symbol": false,
 		"QUnit": false,
 		"ajaxTest": false,
 		"testIframe": false,
-		"testIframeWithCallback": false,
-		"iframeCallback": true,
 		"createDashboardXML": false,
 		"createXMLFragment": false,
 		"moduleTeardown": false,
-		"testFoo": false,
 		"url": false,
-		"t": false,
 		"q": false,
 		"jQuery": true,
 		"sinon": true,

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,9 @@`
`1`	`1`	`{`
`2`	`2`	`"root": true,`
`3`	`3`
`4`		`- "extends": "../.eslintrc-browser.json"`
	`4`	`+ "extends": "../.eslintrc-browser.json",`
	`5`	`+`
	`6`	`+ "globals": {`
	`7`	`+ "jQuery": true`
	`8`	`+ }`
`5`	`9`	`}`