chore: update eslint config (#86)

svg-utils · Nov 16, 2024 · 5327763 · 5327763
1 parent 339f2e3
commit 5327763
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 25 deletions.
diff --git a/eslint.config.mjs → eslint.config.js b/eslint.config.mjs → eslint.config.js
@@ -1,23 +1,10 @@
 import js from '@eslint/js';
 import globals from 'globals';
 
-/**
- * @typedef {import('eslint').Linter.FlatConfig} FlatConfig
- */
-
-/** @type {FlatConfig[]} */
 export default [
+  js.configs.recommended,
   {
-    ignores: [
-      '.yarn/**',
-      '.yarnrc.yml',
-      'node_modules/**',
-      'dist/**',
-      'test/regression-fixtures/**',
-      'test/regression-diffs/**',
-      'test/cli/output/**',
-      'coverage/**',
-    ],
+    ignores: ['dist/**'],
   },
   {
     languageOptions: {
@@ -29,10 +16,7 @@ export default [
     linterOptions: {
       reportUnusedDisableDirectives: 'error',
     },
-  },
-  js.configs.recommended,
-  {
-    files: ['**/*.js', '**/*.mjs'],
+    files: ['**/*.js', '**/*.cjs', '**/*.mjs'],
     rules: {
       strict: 'error',
     },

diff --git a/lib/stop-offset.js b/lib/stop-offset.js
@@ -31,7 +31,7 @@ export class StopOffsetValue extends AttValue {
   }
 
   /**
-   * @param {string|AttValue} value
+   * @param {string|import('./types.js').AttValue} value
    * @returns {StopOffsetValue}
    */
   static getStopOffsetObj(value) {

diff --git a/package.json b/package.json
@@ -75,7 +75,7 @@
     "sax": "^1.4.1"
   },
   "devDependencies": {
-    "@eslint/js": "^9.14.0",
+    "@eslint/js": "^9.15.0",
     "@rollup/plugin-commonjs": "^28.0.1",
     "@rollup/plugin-node-resolve": "^15.3.0",
     "@rollup/plugin-terser": "^0.4.4",

diff --git a/plugins/mergeGradients.js b/plugins/mergeGradients.js
@@ -45,7 +45,7 @@ export const fn = (root, params, info) => {
             return;
         }
 
-        const gradientId = element.attributes.id;
+        const gradientId = element.attributes.id.toString();
         if (!gradientId) {
           return;
         }
@@ -78,7 +78,7 @@ export const fn = (root, params, info) => {
           // Update all references.
 
           addChildToDelete(childrenToDelete, duplicate);
-          const dupId = duplicate.attributes.id;
+          const dupId = duplicate.attributes.id.toString();
           const dupReferencingEls = referencedIds.get(dupId);
           if (!dupReferencingEls) {
             continue;

diff --git a/test/fixtures/config-loader/cjs/svgo.config.cjs b/test/fixtures/config-loader/cjs/svgo.config.cjs
@@ -1,3 +1,5 @@
+'use strict';
+
 module.exports = {
   plugins: ['cjs'],
 };
diff --git a/test/svgo.cjs b/test/svgo.cjs
@@ -1,3 +1,5 @@
+'use strict';
+
 const assert = require('assert');
 const {
   VERSION,

diff --git a/yarn.lock b/yarn.lock
@@ -437,7 +437,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@eslint/js@npm:9.15.0, @eslint/js@npm:^9.14.0":
+"@eslint/js@npm:9.15.0, @eslint/js@npm:^9.15.0":
   version: 9.15.0
   resolution: "@eslint/js@npm:9.15.0"
   checksum: 8b2aa35b62c1969c5e3aa5e33b4e072cc450b95ee9bc08b5a84aa5cbed10a01a9a8f0463b8bb3768ceeb5abfaa82abfb95650061338c3b467835757b62ffc4ba
@@ -4484,7 +4484,7 @@ __metadata:
   version: 0.0.0-use.local
   resolution: "svgo-ll@workspace:."
   dependencies:
-    "@eslint/js": ^9.14.0
+    "@eslint/js": ^9.15.0
     "@rollup/plugin-commonjs": ^28.0.1
     "@rollup/plugin-node-resolve": ^15.3.0
     "@rollup/plugin-terser": ^0.4.4