Update to ESLint 8.57.0

TerriaJS · May 22, 2024 · 91744ef · 91744ef
1 parent e7fdecb
commit 91744ef
Show file tree

Hide file tree

Showing 6 changed files with 192 additions and 205 deletions.
diff --git a/gulpfile.js b/gulpfile.js
@@ -44,19 +44,32 @@ gulp.task("watch-specs", function (done) {
 });
 
 gulp.task("lint", function (done) {
-  var runExternalModule = require("./buildprocess/runExternalModule");
-
-  runExternalModule("eslint/bin/eslint.js", [
-    "lib",
-    "test",
-    "--ext",
-    ".jsx,.js,.ts,.tsx",
-    "--ignore-pattern",
-    "lib/ThirdParty",
-    "--max-warnings",
-    "264" // TODO: Bring this back to 0
-  ]);
-
+  const spawnSync = require("child_process").spawnSync;
+  // If upgrading to ESLint 9 without changing config file format:
+  // process.env.ESLINT_USE_FLAT_CONFIG = "false";
+  const result = spawnSync(
+    "node_modules/eslint/bin/eslint.js",
+    [
+      "lib",
+      "test",
+      "--ext",
+      ".jsx,.js,.ts,.tsx",
+      "--ignore-pattern",
+      "lib/ThirdParty",
+      "--max-warnings",
+      "264" // TODO: Bring this back to 0
+    ],
+    {
+      stdio: "inherit",
+      shell: true
+    }
+  );
+  if (result.status !== 0) {
+    const PluginError = require("plugin-error");
+    throw new PluginError("eslint", "ESLint exited with an error.", {
+      showStack: false
+    });
+  }
   done();
 });
 

diff --git a/lib/Models/Catalog/Ows/WebMapServiceCapabilitiesStratum.ts b/lib/Models/Catalog/Ows/WebMapServiceCapabilitiesStratum.ts
@@ -95,7 +95,8 @@ export default class WebMapServiceCapabilitiesStratum extends LoadableStratum(
     Array.from(this.capabilitiesLayers.values()).forEach((layer) => {
       if (!layer?.MetadataURL) return;
       Array.isArray(layer?.MetadataURL)
-        ? metadataUrls.push(...layer?.MetadataURL)
+        ? // eslint-disable-next-line no-unsafe-optional-chaining
+          metadataUrls.push(...layer?.MetadataURL)
         : metadataUrls.push(layer?.MetadataURL as MetadataURL);
     });
 

diff --git a/lib/Models/Catalog/Ows/WebMapTileServiceCatalogItem.ts b/lib/Models/Catalog/Ows/WebMapTileServiceCatalogItem.ts
@@ -583,6 +583,7 @@ class WebMapTileServiceCatalogItem extends MappableMixin(
     let tileMatrixSetLinks: TileMatrixSetLink[] = [];
     if (layer?.TileMatrixSetLink) {
       if (Array.isArray(layer?.TileMatrixSetLink)) {
+        // eslint-disable-next-line no-unsafe-optional-chaining
         tileMatrixSetLinks = [...layer?.TileMatrixSetLink];
       } else {
         tileMatrixSetLinks = [layer.TileMatrixSetLink];

diff --git a/lib/Models/Workflows/TableStylingWorkflow.ts b/lib/Models/Workflows/TableStylingWorkflow.ts
@@ -1952,6 +1952,7 @@ export default class TableStylingWorkflow
             ),
             isOpen: true,
             selectableDimensions: filterOutUndefined([
+              // eslint-disable-next-line no-unsafe-optional-chaining
               ...traits.enum?.map((enumPoint, idx) => {
                 const dims: SelectableDimensionGroup = {
                   type: "group",

diff --git a/package.json b/package.json
@@ -201,7 +201,7 @@
     "@typescript-eslint/eslint-plugin": "^6.12.0",
     "@typescript-eslint/parser": "^6.12.0",
     "babel-plugin-styled-components": "^1.10.7",
-    "eslint": "^7.20.0",
+    "eslint": "^8.57.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-react": "^7.19.0",
     "eslint-plugin-react-hooks": "^4.6.0",