Simplify dev dependencies and config for ESLint.

jaydenseric · jaydenseric · commit 80cfe716d021 · 2022-02-11T12:57:56.000+11:00
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -1,22 +1,31 @@
 {
-  "extends": ["env"],
-  "settings": {
-    "jsdoc": {
-      "mode": "typescript"
-    },
-    "polyfills": [
-      "AbortController",
-      "CustomEvent",
-      "Event",
-      "EventTarget",
-      "fetch",
-      "FormData",
-      "performance"
-    ]
+  "extends": ["eslint:recommended", "plugin:react-hooks/recommended"],
+  "env": {
+    "es2021": true,
+    "node": true,
+    "browser": true
   },
+  "parserOptions": {
+    "ecmaVersion": "latest"
+  },
+  "plugins": ["simple-import-sort"],
   "rules": {
-    "jsdoc/require-description": "off",
-    "jsdoc/require-returns": "off",
-    "jsdoc/valid-types": "off"
-  }
+    "simple-import-sort/imports": "error",
+    "simple-import-sort/exports": "error"
+  },
+  "overrides": [
+    {
+      "files": ["*.mjs"],
+      "parserOptions": {
+        "sourceType": "module"
+      },
+      "globals": {
+        "__dirname": "off",
+        "__filename": "off",
+        "exports": "off",
+        "module": "off",
+        "require": "off"
+      }
+    }
+  ]
 }
diff --git a/Cache.test.mjs b/Cache.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 import assertInstanceOf from "./test/assertInstanceOf.mjs";
diff --git a/CacheContext.test.mjs b/CacheContext.test.mjs
@@ -3,6 +3,7 @@
 import { strictEqual } from "assert";
 import React from "react";
 import ReactDOMServer from "react-dom/server.js";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/HYDRATION_TIME_MS.test.mjs b/HYDRATION_TIME_MS.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { strictEqual } from "assert";
+
 import HYDRATION_TIME_MS from "./HYDRATION_TIME_MS.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 
diff --git a/HydrationTimeStampContext.test.mjs b/HydrationTimeStampContext.test.mjs
@@ -3,6 +3,7 @@
 import { strictEqual } from "assert";
 import React from "react";
 import ReactDOMServer from "react-dom/server.js";
+
 import HydrationTimeStampContext from "./HydrationTimeStampContext.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 
diff --git a/Loading.test.mjs b/Loading.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual } from "assert";
+
 import Loading from "./Loading.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 import assertInstanceOf from "./test/assertInstanceOf.mjs";
diff --git a/LoadingCacheValue.mjs b/LoadingCacheValue.mjs
@@ -1,8 +1,8 @@
 // @ts-check
 
 import Cache from "./Cache.mjs";
-import Loading from "./Loading.mjs";
 import cacheEntrySet from "./cacheEntrySet.mjs";
+import Loading from "./Loading.mjs";
 
 /** @typedef {import("./Cache.mjs").CacheValue} CacheValue */
 /** @typedef {import("./Cache.mjs").CacheEventMap} CacheEventMap */
diff --git a/LoadingCacheValue.test.mjs b/LoadingCacheValue.test.mjs
@@ -1,13 +1,14 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import Loading from "./Loading.mjs";
 import LoadingCacheValue from "./LoadingCacheValue.mjs";
-import Deferred from "./test/Deferred.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 import assertInstanceOf from "./test/assertInstanceOf.mjs";
 import assertTypeOf from "./test/assertTypeOf.mjs";
+import Deferred from "./test/Deferred.mjs";
 
 /**
  * Adds `LoadingCacheValue` tests.
diff --git a/LoadingContext.test.mjs b/LoadingContext.test.mjs
@@ -3,6 +3,7 @@
 import { strictEqual } from "assert";
 import React from "react";
 import ReactDOMServer from "react-dom/server.js";
+
 import Loading from "./Loading.mjs";
 import LoadingContext from "./LoadingContext.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/Provider.mjs b/Provider.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import React from "react";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
 import HydrationTimeStampContext from "./HydrationTimeStampContext.mjs";
diff --git a/Provider.test.mjs b/Provider.test.mjs
@@ -3,6 +3,7 @@
 import { strictEqual, throws } from "assert";
 import React from "react";
 import ReactTestRenderer from "react-test-renderer";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
 import HydrationTimeStampContext from "./HydrationTimeStampContext.mjs";
diff --git a/cacheDelete.test.mjs b/cacheDelete.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import cacheDelete from "./cacheDelete.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/cacheEntryDelete.test.mjs b/cacheEntryDelete.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import cacheEntryDelete from "./cacheEntryDelete.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/cacheEntryPrune.test.mjs b/cacheEntryPrune.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import cacheEntryPrune from "./cacheEntryPrune.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/cacheEntrySet.test.mjs b/cacheEntrySet.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import cacheEntrySet from "./cacheEntrySet.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/cacheEntryStale.test.mjs b/cacheEntryStale.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import cacheEntryStale from "./cacheEntryStale.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/cachePrune.test.mjs b/cachePrune.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import cachePrune from "./cachePrune.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/cacheStale.test.mjs b/cacheStale.test.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import { deepStrictEqual, strictEqual, throws } from "assert";
+
 import Cache from "./Cache.mjs";
 import cacheStale from "./cacheStale.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/changelog.md b/changelog.md
@@ -5,6 +5,7 @@
 ### Patch
 
 - Updated dev dependencies.
+- Simplified dev dependencies and config for ESLint.
 
 ## 17.0.0
 
diff --git a/fetchGraphQL.test.mjs b/fetchGraphQL.test.mjs
@@ -3,6 +3,7 @@
 import { deepStrictEqual, strictEqual } from "assert";
 import { Response } from "node-fetch";
 import revertableGlobals from "revertable-globals";
+
 import fetchGraphQL from "./fetchGraphQL.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 
diff --git a/fetchOptionsGraphQL.test.mjs b/fetchOptionsGraphQL.test.mjs
@@ -4,6 +4,7 @@ import { deepStrictEqual, strictEqual } from "assert";
 import { File } from "fetch-blob/file.js";
 import { FormData } from "formdata-polyfill/esm.min.js";
 import revertableGlobals from "revertable-globals";
+
 import fetchOptionsGraphQL from "./fetchOptionsGraphQL.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 import assertInstanceOf from "./test/assertInstanceOf.mjs";
diff --git a/package.json b/package.json
@@ -115,15 +115,8 @@
     "coverage-node": "^6.0.1",
     "esbuild": "^0.14.21",
     "eslint": "^8.8.0",
-    "eslint-config-env": "^23.0.2",
-    "eslint-config-prettier": "^8.3.0",
-    "eslint-plugin-compat": "^4.0.2",
-    "eslint-plugin-import": "^2.25.4",
-    "eslint-plugin-jsdoc": "^37.8.2",
-    "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-prettier": "^4.0.0",
-    "eslint-plugin-react": "^7.28.0",
     "eslint-plugin-react-hooks": "^4.3.0",
+    "eslint-plugin-simple-import-sort": "^7.0.0",
     "event-target-shim": "^6.0.2",
     "fetch-blob": "^3.1.4",
     "filter-console": "^1.0.0",
diff --git a/test.mjs b/test.mjs
@@ -1,15 +1,11 @@
 // @ts-check
 
 import "./test/polyfills.mjs";
+
 import TestDirector from "test-director";
+
 import test_Cache from "./Cache.test.mjs";
 import test_CacheContext from "./CacheContext.test.mjs";
-import test_HYDRATION_TIME_MS from "./HYDRATION_TIME_MS.test.mjs";
-import test_HydrationTimeStampContext from "./HydrationTimeStampContext.test.mjs";
-import test_Loading from "./Loading.test.mjs";
-import test_LoadingCacheValue from "./LoadingCacheValue.test.mjs";
-import test_LoadingContext from "./LoadingContext.test.mjs";
-import test_Provider from "./Provider.test.mjs";
 import test_cacheDelete from "./cacheDelete.test.mjs";
 import test_cacheEntryDelete from "./cacheEntryDelete.test.mjs";
 import test_cacheEntryPrune from "./cacheEntryPrune.test.mjs";
@@ -19,18 +15,24 @@ import test_cachePrune from "./cachePrune.test.mjs";
 import test_cacheStale from "./cacheStale.test.mjs";
 import test_fetchGraphQL from "./fetchGraphQL.test.mjs";
 import test_fetchOptionsGraphQL from "./fetchOptionsGraphQL.test.mjs";
+import test_HYDRATION_TIME_MS from "./HYDRATION_TIME_MS.test.mjs";
+import test_HydrationTimeStampContext from "./HydrationTimeStampContext.test.mjs";
+import test_Loading from "./Loading.test.mjs";
+import test_LoadingCacheValue from "./LoadingCacheValue.test.mjs";
+import test_LoadingContext from "./LoadingContext.test.mjs";
+import test_Provider from "./Provider.test.mjs";
 import test_useAutoAbortLoad from "./useAutoAbortLoad.test.mjs";
 import test_useAutoLoad from "./useAutoLoad.test.mjs";
 import test_useCache from "./useCache.test.mjs";
 import test_useCacheEntry from "./useCacheEntry.test.mjs";
 import test_useCacheEntryPrunePrevention from "./useCacheEntryPrunePrevention.test.mjs";
 import test_useForceUpdate from "./useForceUpdate.test.mjs";
 import test_useLoadGraphQL from "./useLoadGraphQL.test.mjs";
+import test_useLoading from "./useLoading.test.mjs";
+import test_useLoadingEntry from "./useLoadingEntry.test.mjs";
 import test_useLoadOnDelete from "./useLoadOnDelete.test.mjs";
 import test_useLoadOnMount from "./useLoadOnMount.test.mjs";
 import test_useLoadOnStale from "./useLoadOnStale.test.mjs";
-import test_useLoading from "./useLoading.test.mjs";
-import test_useLoadingEntry from "./useLoadingEntry.test.mjs";
 import test_useWaterfallLoad from "./useWaterfallLoad.test.mjs";
 
 const tests = new TestDirector();
diff --git a/test/assertBundleSize.mjs b/test/assertBundleSize.mjs
@@ -1,9 +1,9 @@
 // @ts-check
 
 import { fail } from "assert";
-import { fileURLToPath } from "url";
 import esbuild from "esbuild";
 import { gzipSize } from "gzip-size";
+import { fileURLToPath } from "url";
 
 /**
  * Asserts the minified and gzipped bundle size of a module.
diff --git a/test/assertInstanceOf.mjs b/test/assertInstanceOf.mjs
@@ -3,7 +3,6 @@
 import { AssertionError } from "assert";
 import { inspect } from "util";
 
-// eslint-disable-next-line jsdoc/require-returns-check
 /**
  * Asserts a value is an instance of a given class.
  * @template ExpectedClass
diff --git a/test/assertTypeOf.mjs b/test/assertTypeOf.mjs
@@ -3,7 +3,6 @@
 import { AssertionError } from "assert";
 import { inspect } from "util";
 
-// eslint-disable-next-line jsdoc/require-returns-check
 /**
  * Asserts a value is a given type.
  * @template {keyof TypeMap} ExpectedType
diff --git a/useAutoAbortLoad.test.mjs b/useAutoAbortLoad.test.mjs
@@ -1,8 +1,9 @@
 // @ts-check
 
-import { notStrictEqual, strictEqual, throws } from "assert";
 import { cleanup, renderHook } from "@testing-library/react-hooks/lib/pure.js";
+import { notStrictEqual, strictEqual, throws } from "assert";
 import React from "react";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
 import Loading from "./Loading.mjs";
diff --git a/useAutoLoad.test.mjs b/useAutoLoad.test.mjs
@@ -1,15 +1,16 @@
 // @ts-check
 
-import { strictEqual, throws } from "assert";
 import { cleanup, renderHook } from "@testing-library/react-hooks/lib/pure.js";
+import { strictEqual, throws } from "assert";
 import React from "react";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
-import Loading from "./Loading.mjs";
-import LoadingCacheValue from "./LoadingCacheValue.mjs";
 import cacheEntryDelete from "./cacheEntryDelete.mjs";
 import cacheEntryPrune from "./cacheEntryPrune.mjs";
 import cacheEntryStale from "./cacheEntryStale.mjs";
+import Loading from "./Loading.mjs";
+import LoadingCacheValue from "./LoadingCacheValue.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
 import assertTypeOf from "./test/assertTypeOf.mjs";
 import useAutoLoad from "./useAutoLoad.mjs";
diff --git a/useCache.mjs b/useCache.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import React from "react";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
 
diff --git a/useCache.test.mjs b/useCache.test.mjs
@@ -1,12 +1,13 @@
 // @ts-check
 
-import { deepStrictEqual, strictEqual } from "assert";
 import {
   cleanup,
   renderHook,
   suppressErrorOutput,
 } from "@testing-library/react-hooks/lib/pure.js";
+import { deepStrictEqual, strictEqual } from "assert";
 import React from "react";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
 import assertBundleSize from "./test/assertBundleSize.mjs";
diff --git a/useCacheEntry.mjs b/useCacheEntry.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import React from "react";
+
 import useCache from "./useCache.mjs";
 import useForceUpdate from "./useForceUpdate.mjs";
 
diff --git a/useCacheEntry.test.mjs b/useCacheEntry.test.mjs
@@ -1,13 +1,14 @@
 // @ts-check
 
-import { deepStrictEqual, strictEqual, throws } from "assert";
 import {
   act,
   cleanup,
   renderHook,
   suppressErrorOutput,
 } from "@testing-library/react-hooks/lib/pure.js";
+import { deepStrictEqual, strictEqual, throws } from "assert";
 import React from "react";
+
 import Cache from "./Cache.mjs";
 import CacheContext from "./CacheContext.mjs";
 import cacheEntryDelete from "./cacheEntryDelete.mjs";
diff --git a/useCacheEntryPrunePrevention.mjs b/useCacheEntryPrunePrevention.mjs
@@ -1,6 +1,7 @@
 // @ts-check
 
 import React from "react";
+
 import useCache from "./useCache.mjs";
 
 /** @typedef {import("./Cache.mjs").default} Cache */
diff --git a/useCacheEntryPrunePrevention.test.mjs b/useCacheEntryPrunePrevention.test.mjs
diff --git a/useForceUpdate.test.mjs b/useForceUpdate.test.mjs
diff --git a/useLoadGraphQL.mjs b/useLoadGraphQL.mjs
diff --git a/useLoadGraphQL.test.mjs b/useLoadGraphQL.test.mjs
diff --git a/useLoadOnDelete.mjs b/useLoadOnDelete.mjs
diff --git a/useLoadOnDelete.test.mjs b/useLoadOnDelete.test.mjs
diff --git a/useLoadOnMount.mjs b/useLoadOnMount.mjs
diff --git a/useLoadOnMount.test.mjs b/useLoadOnMount.test.mjs
diff --git a/useLoadOnStale.mjs b/useLoadOnStale.mjs
diff --git a/useLoadOnStale.test.mjs b/useLoadOnStale.test.mjs
diff --git a/useLoading.mjs b/useLoading.mjs
diff --git a/useLoading.test.mjs b/useLoading.test.mjs
diff --git a/useLoadingEntry.mjs b/useLoadingEntry.mjs
diff --git a/useLoadingEntry.test.mjs b/useLoadingEntry.test.mjs
diff --git a/useWaterfallLoad.mjs b/useWaterfallLoad.mjs
diff --git a/useWaterfallLoad.test.mjs b/useWaterfallLoad.test.mjs
