Various review feedback for ESLint configuration (#204)

Co-authored-by: Frederik Prijck <[email protected]>

.eslintrc.json

@@ -1,24 +1,27 @@
 {
-  "env": {
-    "node": true,
-    "commonjs": true,
-    "es2017": true,
-    "mocha": true
-  },
-  "ignorePatterns": ["**/*.mjs", "**/*.js"],
+  "ignorePatterns": ["build", "coverage"],
   "extends": [
     "eslint:recommended",
     "plugin:@typescript-eslint/recommended-type-checked",
     "plugin:@typescript-eslint/stylistic-type-checked",
+    "plugin:import/recommended",
+    "plugin:import/typescript",
     "plugin:prettier/recommended"
   ],
-  "plugins": ["@typescript-eslint", "import"],
+  "overrides": [
+    {
+      "files": ["lib/**", "test/**"],
+      "rules": {
+        "import/no-default-export": "error"
+      }
+    }
+  ],
+  "settings": {
+    "import/resolver": {
+      "typescript": true
+    }
+  },
   "parserOptions": {
-    "ecmaVersion": 9,
     "project": true
-  },
-  "rules": {
-    "import/no-default-export": "error",
-    "import/extensions": ["error", "always"]
   }
 }

.prettierrc

@@ -1,4 +1,3 @@
 {
-    "printWidth": 100,
-    "singleQuote": false
-  }
+  "printWidth": 100
+}

jest.config.ts

@@ -8,10 +8,7 @@ const jestConfig: JestConfigWithTsJest = {
   testMatch: ["**/test/**/*.test.ts", "**/test/tests.ts"],
   coverageProvider: "v8",
   coveragePathIgnorePatterns: ["/node_modules/", "./test"],
-  reporters: [
-    "default",
-    ["jest-junit", { outputDirectory: "test-results/jest" }],
-  ],
+  reporters: ["default", ["jest-junit", { outputDirectory: "test-results/jest" }]],
   coverageReporters: ["lcov", "text", "text-summary"],
 };
 

package.json

@@ -34,7 +34,7 @@
     "prebuild": "rimraf build",
     "build": "tsc -b ./tsconfig.cjs.json ./tsconfig.esm.json && echo '{\"type\": \"commonjs\"}'> build/cjs/package.json",
     "build:watch": "tsc -b --watch ./tsconfig.cjs.json ./tsconfig.esm.json && echo '{\"type\": \"commonjs\"}'> build/cjs/package.json",
-    "lint": "eslint ./lib ./test --ext ts",
+    "lint": "eslint .",
     "lint:package": "publint",
     "test": "npm run test:node && npm run test:browser",
     "test:node": "NODE_OPTIONS='--experimental-vm-modules --no-warnings' jest --coverage",
@@ -51,8 +51,9 @@
     "@typescript-eslint/parser": "^6.4.1",
     "browser-sync": "^2.29.3",
     "concurrently": "^8.2.0",
-    "eslint": "^8.47.0",
+    "eslint": "^8.48.0",
     "eslint-config-prettier": "^9.0.0",
+    "eslint-import-resolver-typescript": "^3.6.0",
     "eslint-plugin-import": "^2.28.1",
     "eslint-plugin-prettier": "^5.0.0",
     "husky": "^8.0.3",