eslint: Move max-len disable-directive to dist/.eslintrc.json

edg2s · edg2s · commit d99e7fc0a8dd · 2020-04-21T12:11:41.000+01:00
This disable-directive only applies to the built version, so put it in /dist. This avoids a warning about an unused directive in the source version. Follow-up to jquery#4520
diff --git a/dist/.eslintrc.json b/dist/.eslintrc.json
@@ -20,6 +20,8 @@
 			"rules": {
 				// That is okay for the built version
 				"no-multiple-empty-lines": "off",
+				// Version number substituion may trigger max-len
+				"max-len": "off",
 				"one-var": "off"
 			}
 		}
diff --git a/src/core.js b/src/core.js
@@ -14,8 +14,6 @@ import isWindow from "./var/isWindow.js";
 import DOMEval from "./core/DOMEval.js";
 import toType from "./core/toType.js";
 
-// When custom compilation is used, the version string can get large.
-// eslint-disable-next-line max-len
 var version = "@VERSION",
 
 	rhtmlSuffix = /HTML$/i,

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,8 @@`
`20`	`20`	`"rules": {`
`21`	`21`	`// That is okay for the built version`
`22`	`22`	`"no-multiple-empty-lines": "off",`
	`23`	`+ // Version number substituion may trigger max-len`
	`24`	`+ "max-len": "off",`
`23`	`25`	`"one-var": "off"`
`24`	`26`	`}`
`25`	`27`	`}`