From 629a62066859d77159c3dbe8a2f4120bebb5bf04 Mon Sep 17 00:00:00 2001
From: Ryan Wang <i@ryanc.cc>
Date: Sun, 23 Apr 2023 12:37:33 +0800
Subject: [PATCH] chore: set eslint max-warnings option and remove unused
 imports (#3822)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

#### What type of PR is this?

/kind improvement
/area console

#### What this PR does / why we need it:

调整 ESLint 的执行参数，添加 max-warnings 为 0，意味着在提交或者运行 CI 的时候不再允许包含 warning 级别的检测。

#### Does this PR introduce a user-facing change?

```release-note
None
```
---
 console/package.json                                 | 4 ++--
 console/src/modules/interface/themes/ThemeDetail.vue | 3 ---
 console/src/modules/system/roles/RoleDetail.vue      | 2 --
 3 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/console/package.json b/console/package.json
index 4667f7a30..e0431a24b 100644
--- a/console/package.json
+++ b/console/package.json
@@ -15,7 +15,7 @@
     "test:e2e": "start-server-and-test preview http://127.0.0.1:5050/ 'cypress open'",
     "test:e2e:ci": "start-server-and-test preview http://127.0.0.1:5050/ 'cypress run'",
     "typecheck": "vue-tsc --noEmit -p tsconfig.app.json --composite false && pnpm run typecheck:packages",
-    "lint": "eslint ./src --ext .vue,.js,.jsx,.cjs,.mjs,.ts,.tsx,.cts,.mts --ignore-path .gitignore && pnpm run lint:packages",
+    "lint": "eslint ./src --ext .vue,.js,.jsx,.cjs,.mjs,.ts,.tsx,.cts,.mts --ignore-path .gitignore --max-warnings=0 && pnpm run lint:packages",
     "prettier": "prettier --write './src/**/*.{vue,js,jsx,ts,tsx,css,scss,json,yml,yaml,html}' && pnpm run prettier:packages",
     "typecheck:packages": "pnpm --parallel --filter \"./packages/**\" run typecheck",
     "lint:packages": "pnpm --parallel --filter \"./packages/**\" lint",
@@ -27,7 +27,7 @@
       "prettier --write"
     ],
     "*.{js,ts,vue,tsx,jsx}": [
-      "eslint --fix"
+      "eslint --fix --max-warnings=0"
     ]
   },
   "workspaces": [
diff --git a/console/src/modules/interface/themes/ThemeDetail.vue b/console/src/modules/interface/themes/ThemeDetail.vue
index 6c59ff6d4..b5ab21e33 100644
--- a/console/src/modules/interface/themes/ThemeDetail.vue
+++ b/console/src/modules/interface/themes/ThemeDetail.vue
@@ -1,13 +1,10 @@
 <script lang="ts" setup>
 // core libs
 import { inject, ref } from "vue";
-import { RouterLink } from "vue-router";
 import { useThemeLifeCycle } from "./composables/use-theme";
 
 // components
 import {
-  VAlert,
-  VSpace,
   VTag,
   IconMore,
   Dialog,
diff --git a/console/src/modules/system/roles/RoleDetail.vue b/console/src/modules/system/roles/RoleDetail.vue
index a995cdedc..df69468ae 100644
--- a/console/src/modules/system/roles/RoleDetail.vue
+++ b/console/src/modules/system/roles/RoleDetail.vue
@@ -1,13 +1,11 @@
 <script lang="ts" setup>
 import {
-  IconArrowRight,
   IconShieldUser,
   VButton,
   VCard,
   VPageHeader,
   VTabbar,
   VTag,
-  VAvatar,
   VAlert,
   VDescription,
   VDescriptionItem,