update to ESLint 9

Author: Devin McIntyre

.eslintrc.js

@@ -1,2 +1,2 @@
 # Auto detect text files and perform LF normalization
-* text=auto
+* text eol=lf

.gitattributes

@@ -2,3 +2,6 @@
 npm-debug.log
 package-lock.json
 yarn.lock
+.idea
+eslint-report.html
+/test

.gitignore

@@ -0,0 +1,31 @@
+import { promisify } from "node:util";
+import chip from "node:child_process";
+import fs from "node:fs";
+import path from "node:path";
+
+const exec = promisify( chip.exec );
+
+await exec( "npm install --no-save jquery" );
+
+const fixtures = path.join( ".", "test", "fixtures" );
+
+try {
+	await fs.promises.mkdir( path.join( ".", "test" ) );
+	await fs.promises.mkdir( fixtures );
+} catch ( _ ) {
+	await fs.promises.rm(
+		fixtures,
+		{
+			recursive: true
+		}
+	);
+
+}
+
+await fs.promises.cp(
+    path.join( ".", "node_modules", "jquery", "src" ),
+    path.join( fixtures, "src" ),
+    {
+        recursive: true
+    }
+);

copy-fixtures.js

@@ -0,0 +1,14 @@
+import jquery from "./index.js";
+import globals from "globals";
+
+export default [
+    {
+        "languageOptions": {
+            "globals": {
+                ...globals.amd,
+                ...globals.browser
+            }
+        }
+    },
+    jquery
+];

eslint.config.js

@@ -1,30 +1,37 @@
-module.exports = {
+
+import stylistic from "@stylistic/eslint-plugin-js";
+
+export default {
+	plugins: {
+		"@stylistic/js": stylistic
+	},
 	rules: {
-		"no-negated-in-lhs": "error",
+		"no-unsafe-negation": "error",
 		"no-cond-assign": [ "error", "except-parens" ],
-		curly: [ "error", "all" ],
-		"object-curly-spacing": [ "error", "always" ],
-		"computed-property-spacing": [ "error", "always" ],
-		"array-bracket-spacing": [ "error", "always" ],
-		eqeqeq: [ "error", "smart" ],
+		"curly": [ "error", "all" ],
+		"@stylistic/js/object-curly-spacing": [ "error", "always" ],
+		"@stylistic/js/computed-property-spacing": [ "error", "always" ],
+		"@stylistic/js/array-bracket-spacing": [ "error", "always" ],
+		"eqeqeq": [ "error", "smart" ],
 		"no-unused-expressions": "error",
 		"no-sequences": "error",
 		"no-nested-ternary": "error",
 		"no-unreachable": "error",
-		"wrap-iife": [ "error", "inside" ],
+		"@stylistic/js/wrap-iife": [ "error", "inside" ],
 		"no-caller": "error",
-		quotes: [ "error", "double" ],
+		"@stylistic/js/quotes": [ "error", "double" ],
 		"no-undef": "error",
 		"no-unused-vars": [
 			"error",
 			{
 				args: "all",
-				argsIgnorePattern: "^_"
+				argsIgnorePattern: "^_",
+				caughtErrors: "none"
 			}
 		],
-		"operator-linebreak": [ "error", "after" ],
-		"comma-style": [ "error", "last" ],
-		camelcase: [
+		"@stylistic/js/operator-linebreak": [ "error", "after" ],
+		"@stylistic/js/comma-style": [ "error", "last" ],
+		"camelcase": [
 			"error",
 			{
 				properties: "never"
@@ -36,7 +43,7 @@ module.exports = {
 				allowPattern: "^[a-z]+(_[a-z]+)+$"
 			}
 		],
-		"max-len": [
+		"@stylistic/js/max-len": [
 			"error",
 			{
 				code: 100,
@@ -45,24 +52,24 @@ module.exports = {
 				ignoreRegExpLiterals: true
 			}
 		],
-		"no-mixed-spaces-and-tabs": "error",
-		"no-trailing-spaces": "error",
+		"@stylistic/js/no-mixed-spaces-and-tabs": "error",
+		"@stylistic/js/no-trailing-spaces": "error",
 		"no-irregular-whitespace": "error",
 		"no-multi-str": "error",
-		"comma-dangle": [ "error", "never" ],
-		"comma-spacing": [
+		"@stylistic/js/comma-dangle": [ "error", "never" ],
+		"@stylistic/js/comma-spacing": [
 			"error",
 			{
 				before: false,
 				after: true
 			}
 		],
-		"space-before-blocks": [ "error", "always" ],
-		"space-in-parens": [ "error", "always" ],
-		"keyword-spacing": [ 2 ],
-		"template-curly-spacing": [ "error", "always" ],
-		semi: [ "error", "always" ],
-		"semi-spacing": [
+		"@stylistic/js/space-before-blocks": [ "error", "always" ],
+		"@stylistic/js/space-in-parens": [ "error", "always" ],
+		"@stylistic/js/keyword-spacing": [ 2 ],
+		"@stylistic/js/template-curly-spacing": [ "error", "always" ],
+		"@stylistic/js/semi": [ "error", "always" ],
+		"@stylistic/js/semi-spacing": [
 			"error",
 			{
 
@@ -71,35 +78,35 @@ module.exports = {
 				after: true
 			}
 		],
-		"no-extra-semi": "error",
-		"space-infix-ops": "error",
-		"eol-last": "error",
-		"lines-around-comment": [
+		"@stylistic/js/no-extra-semi": "error",
+		"@stylistic/js/space-infix-ops": "error",
+		"@stylistic/js/eol-last": "error",
+		"@stylistic/js/lines-around-comment": [
 			"error",
 			{
 				beforeLineComment: true
 			}
 		],
-		"linebreak-style": [ "error", "unix" ],
+		"@stylistic/js/linebreak-style": [ "error", "unix" ],
 		"no-with": "error",
-		"brace-style": "error",
-		"space-before-function-paren": [ "error", "never" ],
+		"@stylistic/js/brace-style": "error",
+		"@stylistic/js/space-before-function-paren": [ "error", "never" ],
 		"no-loop-func": "error",
-		"no-spaced-func": "error",
-		"key-spacing": [
+		"@stylistic/js/function-call-spacing": [ "error", "never" ],
+		"@stylistic/js/key-spacing": [
 			"error",
 			{
 				beforeColon: false,
 				afterColon: true
 			}
 		],
-		"space-unary-ops": [
+		"@stylistic/js/space-unary-ops": [
 			"error",
 			{
 				words: false,
 				nonwords: false
 			}
 		],
-		"no-multiple-empty-lines": 2
+		"@stylistic/js/no-multiple-empty-lines": 2
 	}
 };

index.js

@@ -1,10 +1,13 @@
 {
   "name": "eslint-config-jquery",
-  "version": "3.0.2",
+  "version": "4.0.0",
   "description": "jQuery eslint config",
   "main": "index.js",
+  "type": "module",
   "scripts": {
-    "test": "eslint ."
+    "test": "npm run copy-fixtures && npm run lint",
+    "lint": "eslint",
+    "copy-fixtures": "node copy-fixtures.js"
   },
   "repository": {
     "type": "git",
@@ -25,7 +28,13 @@
   ],
   "author": "Gaidarenko Oleg <markelog@gmail.com>",
   "license": "MIT",
+  "dependencies": {
+    "@stylistic/eslint-plugin-js": "^2.9.0"
+  },
+  "peerDependencies": {
+    "eslint": ">=9.0.0"
+  },
   "devDependencies": {
-    "eslint": "^8.5.0"
+    "eslint": "^9.12.0"
   }
 }

package.json

@@ -6,11 +6,19 @@
 npm install --save-dev eslint-config-jquery
 ```
 
-Configure ESLint with a `.eslintrc` file using the following contents:
-```json
-{
-	"extends": "jquery"
-}
+Version 4.0.0 and above work for ESLint version 9.0.0 and above. Use version 3.0.2 for older ESLint versions.
+
+In your eslint.config file, import it and add it to your flat config array.
+
+```javascript
+import jquery from "eslint-config-jquery";
+
+export default [
+    // ... any other configurations,
+    jquery,
+    // ... any other configurations
+];
+
 ```
 
 ## Status
@@ -20,3 +28,11 @@ This config follows the spirit of the jQuery [code style](https://contribute.jqu
 ## Semver policy
 
 Same approach as in ESLint, see https://github.com/eslint/eslint#user-content-semantic-versioning-policy.
+
+## Testing
+
+To avoid a circular dependency, testing copies JQuery source code into the test directory before running linting for
+this project. 
+
+Note: JQuery 3.7.1 has an invalid eslint ignore statement that makes the tests fail for this project, but that must be 
+fixed in JQuery itself.