From 67a0f86ddeab26a5a0075f34c759276d9b5698bd Mon Sep 17 00:00:00 2001 From: regexowl Date: Fri, 25 Jul 2025 13:51:11 +0200 Subject: [PATCH] ESLint: Set `no-unused-vars` rule to error This rule should output error instead of just a warning. --- eslint.config.js | 8 +++++++- src/Components/CloudProviderConfig/AWSConfig.tsx | 2 +- src/test/mocks/cockpit/cockpitFile.ts | 2 +- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/eslint.config.js b/eslint.config.js index cb0480e1..269a5115 100644 --- a/eslint.config.js +++ b/eslint.config.js @@ -111,7 +111,13 @@ module.exports = defineConfig([ '@typescript-eslint/no-require-imports': 'error', 'disable-autofix/@typescript-eslint/no-unnecessary-condition': 'warn', 'no-unused-vars': 'off', // disable js rule in favor of @typescript-eslint's rule - '@typescript-eslint/no-unused-vars': 'warn', + '@typescript-eslint/no-unused-vars': [ + 'error', + { + argsIgnorePattern: '^_', + varsIgnorePattern: '^_', + }, + ], 'jsx-a11y/no-autofocus': 'off', }, settings: { diff --git a/src/Components/CloudProviderConfig/AWSConfig.tsx b/src/Components/CloudProviderConfig/AWSConfig.tsx index 076ccc5f..4bd2404d 100644 --- a/src/Components/CloudProviderConfig/AWSConfig.tsx +++ b/src/Components/CloudProviderConfig/AWSConfig.tsx @@ -1,4 +1,4 @@ -import React, { useEffect, useState } from 'react'; +import React from 'react'; import { Button, diff --git a/src/test/mocks/cockpit/cockpitFile.ts b/src/test/mocks/cockpit/cockpitFile.ts index b8bbe452..612d9f7e 100644 --- a/src/test/mocks/cockpit/cockpitFile.ts +++ b/src/test/mocks/cockpit/cockpitFile.ts @@ -31,7 +31,7 @@ export const getLastBlueprintReq = () => { return lastRequest.blueprints[lastRequest.blueprints.length - 1]; }; -export const cockpitFile = (filepath: string, options?: object) => { +export const cockpitFile = (filepath: string, _options?: object) => { return { read: (): Promise => { const file = path.parse(filepath);