Build:Event: Make sure all source modules' exports are used (#4648)

mgol · web-flow · commit 40c3abd0ab04 · 2020-06-02T13:45:08.000+02:00
To achieve that, use `eslint-plugin-import`'s `no-unused-modules` rule.

Also, explicitly import `event/trigger.js` from `jquery.js`; so far it was
only imported from ajax.js, making it mistakenly skipped in the
`custom:slim,-deprecated` build.
diff --git a/src/.eslintrc.json b/src/.eslintrc.json
@@ -13,6 +13,10 @@
 	"rules": {
 		"import/extensions": [ "error", "always" ],
 		"import/no-cycle": "error",
+		"import/no-unused-modules": [ "error", {
+			"unusedExports": true,
+			"ignoreExports": [ "src/*.js" ]
+		} ],
 		"indent": [ "error", "tab", {
 			"outerIIFEBody": 0
 		} ]
diff --git a/src/core/init.js b/src/core/init.js
@@ -118,5 +118,3 @@ init.prototype = jQuery.fn;
 
 // Initialize central reference
 rootjQuery = jQuery( document );
-
-export default init;
diff --git a/src/core/parseXML.js b/src/core/parseXML.js
@@ -20,5 +20,3 @@ jQuery.parseXML = function( data ) {
 	}
 	return xml;
 };
-
-export default jQuery.parseXML;
diff --git a/src/core/var/rhtml.js b/src/core/var/rhtml.js
diff --git a/src/event/trigger.js b/src/event/trigger.js
@@ -190,5 +190,3 @@ jQuery.fn.extend( {
 		}
 	}
 } );
-
-export default jQuery;
diff --git a/src/jquery.js b/src/jquery.js
@@ -11,6 +11,7 @@ import "./queue.js";
 import "./queue/delay.js";
 import "./attributes.js";
 import "./event.js";
+import "./event/trigger.js";
 import "./manipulation.js";
 import "./manipulation/_evalUrl.js";
 import "./wrap.js";
diff --git a/src/manipulation/_evalUrl.js b/src/manipulation/_evalUrl.js
@@ -22,5 +22,3 @@ jQuery._evalUrl = function( url, options, doc ) {
 		}
 	} );
 };
-
-export default jQuery._evalUrl;

Original file line number	Diff line number	Diff line change
`@@ -20,5 +20,3 @@ jQuery.parseXML = function( data ) {`
`20`	`20`	`}`
`21`	`21`	`return xml;`
`22`	`22`	`};`
`23`		`-`
`24`		`-export default jQuery.parseXML;`
Original file line number	Diff line number	Diff line change
`@@ -190,5 +190,3 @@ jQuery.fn.extend( {`
`190`	`190`	`}`
`191`	`191`	`}`
`192`	`192`	`} );`
`193`		`-`
`194`		`-export default jQuery;`
Original file line number	Diff line number	Diff line change
`@@ -22,5 +22,3 @@ jQuery._evalUrl = function( url, options, doc ) {`
`22`	`22`	`}`
`23`	`23`	`} );`
`24`	`24`	`};`
`25`		`-`
`26`		`-export default jQuery._evalUrl;`