chore: test new lintstaged rules (heroui-inc#1050)

.eslintrc.json

@@ -11,7 +11,7 @@
     "plugin:react-hooks/recommended",
     "plugin:jsx-a11y/recommended"
   ],
-  "plugins": ["react", "prettier", "import", "@typescript-eslint", "jsx-a11y"],
+  "plugins": ["react", "unused-imports", "import", "@typescript-eslint", "jsx-a11y", "prettier"],
   "parser": "@typescript-eslint/parser",
   "parserOptions": {
     "ecmaFeatures": {
@@ -34,6 +34,9 @@
     "jsx-a11y/click-events-have-key-events": "warn",
     "jsx-a11y/interactive-supports-focus": "warn",
     "prettier/prettier": "warn",
+    "no-unused-vars": "off",
+    "unused-imports/no-unused-vars": "off",
+    "unused-imports/no-unused-imports": "warn",
     "@typescript-eslint/no-unused-vars": [
       "warn",
       {

.lintstagedrc.cjs

@@ -13,16 +13,14 @@ const removeIgnoredFiles = async (files) => {
 };
 
 module.exports = {
-  // *.!(js|ts|jsx|tsx|d.ts)
-  "./packages/**/**/*.{js,cjs,mjs,ts,jsx,tsx,json,md}": async (files) => {
+  "**/*.{cjs,mjs,js,ts,jsx,tsx}": async (files) => {
     const filesToLint = await removeIgnoredFiles(files);
 
-    return [`prettier --config .prettierrc.json --ignore-path --write ${filesToLint}`];
+    return [`eslint -c .eslintrc.json --max-warnings=0 --fix ${filesToLint}`];
   },
-  // TODO: fix linter rules
-  // "./packages/**/**/*.{js,cjs,mjs,ts,jsx,tsx}": async (files) => {
-  //   const filesToLint = await removeIgnoredFiles(files);
+  "**/*.{css,json,md}": async (files) => {
+    const filesToLint = await removeIgnoredFiles(files);
 
-  //   return [`eslint -c .eslintrc.json --max-warnings=0 --fix ${filesToLint}`];
-  // },
+    return [`prettier --config .prettierrc.json --ignore-path --write ${filesToLint}`];
+  },
 };

apps/docs/.eslintrc.json

@@ -1,35 +1,31 @@
 {
-    "extends": ["../../.eslintrc.json"],
-    "ignorePatterns": ["!**/*"],
-    "overrides": [
-      {
-        "files": ["*.ts", "*.tsx", "*.js", "*.jsx"],
-        "parserOptions": {
-            "project": ["apps/docs/tsconfig(.*)?.json"],
-            "ecmaFeatures": {
-              "jsx": true
-            }
-        },
-        "rules": {
-          "react/no-unknown-property": [
-            2,
-            {
-              "ignore": [
-                "jsx",
-                "global"
-              ]
-            }
-          ]
+  "extends": ["../../.eslintrc.json"],
+  "ignorePatterns": ["!**/*"],
+  "overrides": [
+    {
+      "files": ["*.ts", "*.tsx", "*.js", "*.jsx"],
+      "parserOptions": {
+        "project": ["apps/docs/tsconfig(.*)?.json"],
+        "ecmaFeatures": {
+          "jsx": true
         }
       },
-      {
-        "files": ["*.ts", "*.tsx"],
-        "rules": {}
-      },
-      {
-        "files": ["*.js", "*.jsx"],
-        "rules": {}
+      "rules": {
+        "react/no-unknown-property": [
+          2,
+          {
+            "ignore": ["jsx", "global"]
+          }
+        ]
       }
-    ]
-  }
-
+    },
+    {
+      "files": ["*.ts", "*.tsx"],
+      "rules": {}
+    },
+    {
+      "files": ["*.js", "*.jsx"],
+      "rules": {}
+    }
+  ]
+}

package.json

@@ -97,6 +97,7 @@
     "eslint-plugin-promise": "^6.0.0",
     "eslint-plugin-react": "^7.23.2",
     "eslint-plugin-react-hooks": "^4.6.0",
+    "eslint-plugin-unused-imports": "^2.0.0",
     "execa": "^5.1.1",
     "find-up": "^6.3.0",
     "fs-extra": "^10.0.0",

packages/components/avatar/src/avatar.tsx

@@ -2,8 +2,8 @@ import {forwardRef} from "@nextui-org/system";
 import {__DEV__} from "@nextui-org/shared-utils";
 import {useMemo} from "react";
 
-import {useAvatar, UseAvatarProps} from "./use-avatar";
 import {AvatarIcon} from "./avatar-icon";
+import {useAvatar, UseAvatarProps} from "./use-avatar";
 
 export interface AvatarProps extends UseAvatarProps {}
 

packages/components/checkbox/__tests__/checkbox.test.tsx

@@ -2,7 +2,7 @@ import * as React from "react";
 import {render} from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 
-import {Checkbox, CheckboxProps} from "../src";
+import {Checkbox} from "../src";
 
 describe("Checkbox", () => {
   it("should render correctly", () => {