@will-stone/eslint-config 0.6.0 → 0.7.1

package/README.md

@@ -1,5 +1,7 @@
 # @will-stone/eslint-config
 
+My personal [ESLint](https://eslint.org/) config.
+
 ## Usage
 
 ### Install
@@ -55,8 +57,7 @@ the following to your `package.json`:
     "prepare": "husky install"
   },
   "lint-staged": {
-    "*.{js,jsx,ts,tsx}": ["eslint --fix"],
-    "*.{css,json,md}": ["prettier --write"]
+    "*.{js,jsx,ts,tsx}": ["eslint --fix"]
   }
 }
 ```

package/dist/index.cjs

@@ -179,9 +179,7 @@ function base() {
         "camelcase": "error",
         // Write comments however you like
         "capitalized-comments": "off",
-        // If a class method does not use this,
-        // it can sometimes be made into a static function
-        "class-methods-use-this": "error",
+        "class-methods-use-this": "off",
         // Measures the number of linearly independent paths through a program's
         // source code. Rubbish. Use your own judgement on complexity.
         "complexity": "off",
@@ -478,6 +476,7 @@ function base() {
         "no-nonoctal-decimal-escape": "error",
         // This rule disallows calling the Math, JSON, Reflect and Atomics objects as functions.
         "no-obj-calls": "error",
+        "no-object-constructor": "error",
         // the leading zero which identifies an octal literal has been a source of
         // confusion and error in JavaScript code, ECMAScript 5 deprecates the use
         // of octal numeric literals.
@@ -774,13 +773,13 @@ function ignores() {
 }
 
 // src/configs/imports.ts
-var import_eslint_plugin_i = __toESM(require("eslint-plugin-i"), 1);
+var pluginImport = __toESM(require("eslint-plugin-i"), 1);
 var import_eslint_plugin_simple_import_sort = __toESM(require("eslint-plugin-simple-import-sort"), 1);
 function imports() {
   return [
     {
       plugins: {
-        "import": import_eslint_plugin_i.default,
+        "import": pluginImport,
         "simple-import-sort": import_eslint_plugin_simple_import_sort.default
       },
       rules: {
@@ -1246,26 +1245,9 @@ function react() {
         "react/jsx-props-no-multi-spaces": "off",
         "react/jsx-tag-spacing": "off",
         "react/jsx-wrap-multilines": "off",
-        // It is common to use these lifecycle methods without using "this".
-        "class-methods-use-this": [
-          "error",
-          {
-            exceptMethods: [
-              "render",
-              "getInitialState",
-              "getDefaultProps",
-              "getChildContext",
-              "componentWillMount",
-              "componentDidMount",
-              "componentWillReceiveProps",
-              "shouldComponentUpdate",
-              "componentWillUpdate",
-              "componentDidUpdate",
-              "componentWillUnmount",
-              "componentDidCatch"
-            ]
-          }
-        ],
+        /**
+         * React
+         */
         "react/boolean-prop-naming": "error",
         "react/button-has-type": "error",
         "react/default-props-match-prop-types": "error",
@@ -1381,6 +1363,9 @@ function react() {
         "react/static-property-placement": "error",
         "react/style-prop-object": "error",
         "react/void-dom-elements-no-children": "error",
+        /**
+         * Hooks
+         */
         // Checks effect dependencies
         "react-hooks/exhaustive-deps": "warn",
         // Checks rules of Hooks
@@ -1576,6 +1561,7 @@ function typescript() {
         /**
          * Superseded by TS rules below
          */
+        "class-methods-use-this": "off",
         "default-param-last": "off",
         "lines-between-class-members": "off",
         "no-duplicate-imports": "off",
@@ -1599,6 +1585,7 @@ function typescript() {
         "@typescript-eslint/ban-tslint-comment": "warn",
         // Ensures that literals on classes are exposed in a consistent style
         "@typescript-eslint/class-literal-property-style": "warn",
+        "@typescript-eslint/class-methods-use-this": "off",
         // https://typescript-eslint.io/rules/consistent-generic-constructors/
         "@typescript-eslint/consistent-generic-constructors": "warn",
         // Enforce the use of the record type
@@ -2024,22 +2011,23 @@ function unicorn() {
 
 // src/utils.ts
 var import_node_fs = require("fs");
-var import_globby = require("globby");
-var import_load_json_file = require("load-json-file");
+var import_glob = require("glob");
+function loadJsonFileSync(filePath) {
+  const buffer = (0, import_node_fs.readFileSync)(filePath);
+  const data = new TextDecoder().decode(buffer);
+  return JSON.parse(data);
+}
 function checkEnvironment() {
   const isGitIgnore = (0, import_node_fs.existsSync)(".gitignore");
   let isNodeEngine = false;
   let isPrettier = false;
   let isTailwind = false;
   let testingFramework = null;
-  const allPackageJsonPaths = (0, import_globby.globbySync)([
-    "**/package.json",
-    // Could use the `gitIgnore` option but the negative pattern is about 8 times faster.
-    // https://github.com/sindresorhus/globby/issues/50
-    "!**/node_modules"
-  ]);
+  const allPackageJsonPaths = (0, import_glob.globSync)("**/package.json", {
+    ignore: "**/node_modules/**"
+  });
   for (const packageJsonPath of allPackageJsonPaths) {
-    const packageJson = (0, import_load_json_file.loadJsonFileSync)(packageJsonPath);
+    const packageJson = loadJsonFileSync(packageJsonPath);
     if (!testingFramework && Boolean(
       packageJson.dependencies?.jest || packageJson.devDependencies?.jest
     )) {

package/dist/index.js

@@ -143,9 +143,7 @@ function base() {
         "camelcase": "error",
         // Write comments however you like
         "capitalized-comments": "off",
-        // If a class method does not use this,
-        // it can sometimes be made into a static function
-        "class-methods-use-this": "error",
+        "class-methods-use-this": "off",
         // Measures the number of linearly independent paths through a program's
         // source code. Rubbish. Use your own judgement on complexity.
         "complexity": "off",
@@ -442,6 +440,7 @@ function base() {
         "no-nonoctal-decimal-escape": "error",
         // This rule disallows calling the Math, JSON, Reflect and Atomics objects as functions.
         "no-obj-calls": "error",
+        "no-object-constructor": "error",
         // the leading zero which identifies an octal literal has been a source of
         // confusion and error in JavaScript code, ECMAScript 5 deprecates the use
         // of octal numeric literals.
@@ -738,7 +737,7 @@ function ignores() {
 }
 
 // src/configs/imports.ts
-import pluginImport from "eslint-plugin-i";
+import * as pluginImport from "eslint-plugin-i";
 import pluginSimpleImport from "eslint-plugin-simple-import-sort";
 function imports() {
   return [
@@ -1210,26 +1209,9 @@ function react() {
         "react/jsx-props-no-multi-spaces": "off",
         "react/jsx-tag-spacing": "off",
         "react/jsx-wrap-multilines": "off",
-        // It is common to use these lifecycle methods without using "this".
-        "class-methods-use-this": [
-          "error",
-          {
-            exceptMethods: [
-              "render",
-              "getInitialState",
-              "getDefaultProps",
-              "getChildContext",
-              "componentWillMount",
-              "componentDidMount",
-              "componentWillReceiveProps",
-              "shouldComponentUpdate",
-              "componentWillUpdate",
-              "componentDidUpdate",
-              "componentWillUnmount",
-              "componentDidCatch"
-            ]
-          }
-        ],
+        /**
+         * React
+         */
         "react/boolean-prop-naming": "error",
         "react/button-has-type": "error",
         "react/default-props-match-prop-types": "error",
@@ -1345,6 +1327,9 @@ function react() {
         "react/static-property-placement": "error",
         "react/style-prop-object": "error",
         "react/void-dom-elements-no-children": "error",
+        /**
+         * Hooks
+         */
         // Checks effect dependencies
         "react-hooks/exhaustive-deps": "warn",
         // Checks rules of Hooks
@@ -1540,6 +1525,7 @@ function typescript() {
         /**
          * Superseded by TS rules below
          */
+        "class-methods-use-this": "off",
         "default-param-last": "off",
         "lines-between-class-members": "off",
         "no-duplicate-imports": "off",
@@ -1563,6 +1549,7 @@ function typescript() {
         "@typescript-eslint/ban-tslint-comment": "warn",
         // Ensures that literals on classes are exposed in a consistent style
         "@typescript-eslint/class-literal-property-style": "warn",
+        "@typescript-eslint/class-methods-use-this": "off",
         // https://typescript-eslint.io/rules/consistent-generic-constructors/
         "@typescript-eslint/consistent-generic-constructors": "warn",
         // Enforce the use of the record type
@@ -1987,21 +1974,22 @@ function unicorn() {
 }
 
 // src/utils.ts
-import { existsSync } from "fs";
-import { globbySync } from "globby";
-import { loadJsonFileSync } from "load-json-file";
+import { existsSync, readFileSync } from "fs";
+import { globSync } from "glob";
+function loadJsonFileSync(filePath) {
+  const buffer = readFileSync(filePath);
+  const data = new TextDecoder().decode(buffer);
+  return JSON.parse(data);
+}
 function checkEnvironment() {
   const isGitIgnore = existsSync(".gitignore");
   let isNodeEngine = false;
   let isPrettier = false;
   let isTailwind = false;
   let testingFramework = null;
-  const allPackageJsonPaths = globbySync([
-    "**/package.json",
-    // Could use the `gitIgnore` option but the negative pattern is about 8 times faster.
-    // https://github.com/sindresorhus/globby/issues/50
-    "!**/node_modules"
-  ]);
+  const allPackageJsonPaths = globSync("**/package.json", {
+    ignore: "**/node_modules/**"
+  });
   for (const packageJsonPath of allPackageJsonPaths) {
     const packageJson = loadJsonFileSync(packageJsonPath);
     if (!testingFramework && Boolean(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@will-stone/eslint-config",
-  "version": "0.6.0",
+  "version": "0.7.1",
   "description": "Will Stone's ESLint config",
   "keywords": [
     "eslint-config"
@@ -61,6 +61,7 @@
     "eslint-plugin-switch-case": "^1.1.2",
     "eslint-plugin-tailwindcss": "^3.13.0",
     "eslint-plugin-unicorn": "^48.0.1",
+    "glob": "^10.3.10",
     "globals": "^13.22.0",
     "globby": "^13.2.2",
     "load-json-file": "^7.0.1"
@@ -72,7 +73,7 @@
     "@types/node": "^20.8.2",
     "@will-stone/prettier-config": "^7.0.1",
     "eslint": "^8.50.0",
-    "eslint-flat-config-viewer": "^0.0.8",
+    "eslint-flat-config-viewer": "^0.0.9",
     "husky": "^8.0.3",
     "lint-staged": "^14.0.1",
     "prettier": "^3.0.3",