From 760dd24ea006f30859607f27fee91335d507951a Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Sun, 20 Oct 2024 15:05:49 +0200 Subject: [PATCH 1/2] Remove the `eslint-plugin-{fetch-options,html}` dependencies The plugins have originally been introduced in commit d63da81 for the `eslint-plugin-mozilla` dependency, but the `eslint-plugin-mozilla` plugin got removed in commit be93d53 and we also don't use the plugins ourselves in e.g. our `.eslintrc` files (as evidenced by `npx gulp lint` not failing while it does fail if we remove any of the other plugins). --- package-lock.json | 24 ------------------------ package.json | 2 -- 2 files changed, 26 deletions(-) diff --git a/package-lock.json b/package-lock.json index 2a2d36b9c51c2..bb765ca4d7a1c 100644 --- a/package-lock.json +++ b/package-lock.json @@ -22,8 +22,6 @@ "core-js": "^3.38.1", "eslint": "^8.57.1", "eslint-config-prettier": "^9.1.0", - "eslint-plugin-fetch-options": "^0.0.5", - "eslint-plugin-html": "^8.1.2", "eslint-plugin-import": "^2.31.0", "eslint-plugin-jasmine": "^4.2.2", "eslint-plugin-json": "^3.1.0", @@ -5388,28 +5386,6 @@ "ms": "^2.1.1" } }, - "node_modules/eslint-plugin-fetch-options": { - "version": "0.0.5", - "resolved": "https://registry.npmjs.org/eslint-plugin-fetch-options/-/eslint-plugin-fetch-options-0.0.5.tgz", - "integrity": "sha512-ZMxrccsOAZ7uMQ4nMvPJLqLg6oyLF96YOEwTKWAIbDHpwWUp1raXALZom8ikKucaEnhqWSRuBWI8jBXveFwkJg==", - "dev": true, - "engines": { - "node": ">=0.9.0" - } - }, - "node_modules/eslint-plugin-html": { - "version": "8.1.2", - "resolved": "https://registry.npmjs.org/eslint-plugin-html/-/eslint-plugin-html-8.1.2.tgz", - "integrity": "sha512-pbRchDV2SmqbCi/Ev/q3aAikzG9BcFe0IjjqjtMn8eTLq71ZUggyJB6CDmuwGAXmYZHrXI12XTfCqvgcnPRqGw==", - "dev": true, - "license": "ISC", - "dependencies": { - "htmlparser2": "^9.1.0" - }, - "engines": { - "node": ">=16.0.0" - } - }, "node_modules/eslint-plugin-import": { "version": "2.31.0", "resolved": "https://registry.npmjs.org/eslint-plugin-import/-/eslint-plugin-import-2.31.0.tgz", diff --git a/package.json b/package.json index 9e2ebc41e5039..aa224d19a7efc 100644 --- a/package.json +++ b/package.json @@ -17,8 +17,6 @@ "core-js": "^3.38.1", "eslint": "^8.57.1", "eslint-config-prettier": "^9.1.0", - "eslint-plugin-fetch-options": "^0.0.5", - "eslint-plugin-html": "^8.1.2", "eslint-plugin-import": "^2.31.0", "eslint-plugin-jasmine": "^4.2.2", "eslint-plugin-json": "^3.1.0", From 6def1c8453c93aedaca20fb9b40340280ac8a2e6 Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Sun, 20 Oct 2024 15:25:24 +0200 Subject: [PATCH 2/2] Remove the `eslint-config-prettier` dependency The `eslint-plugin-prettier` dependency already installs this dependency by default nowadays, so we don't have to specify it ourselves anymore. --- package-lock.json | 3 ++- package.json | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/package-lock.json b/package-lock.json index bb765ca4d7a1c..647042cbf17a7 100644 --- a/package-lock.json +++ b/package-lock.json @@ -21,7 +21,6 @@ "canvas": "^3.0.0-rc2", "core-js": "^3.38.1", "eslint": "^8.57.1", - "eslint-config-prettier": "^9.1.0", "eslint-plugin-import": "^2.31.0", "eslint-plugin-jasmine": "^4.2.2", "eslint-plugin-json": "^3.1.0", @@ -5331,6 +5330,8 @@ "resolved": "https://registry.npmjs.org/eslint-config-prettier/-/eslint-config-prettier-9.1.0.tgz", "integrity": "sha512-NSWl5BFQWEPi1j4TjVNItzYV7dZXZ+wP6I6ZhrBGpChQhZRUaElihE9uRRkcbRnNb76UMKDF3r+WTmNcGPKsqw==", "dev": true, + "optional": true, + "peer": true, "bin": { "eslint-config-prettier": "bin/cli.js" }, diff --git a/package.json b/package.json index aa224d19a7efc..2414e6c71b8c1 100644 --- a/package.json +++ b/package.json @@ -16,7 +16,6 @@ "canvas": "^3.0.0-rc2", "core-js": "^3.38.1", "eslint": "^8.57.1", - "eslint-config-prettier": "^9.1.0", "eslint-plugin-import": "^2.31.0", "eslint-plugin-jasmine": "^4.2.2", "eslint-plugin-json": "^3.1.0",