eslint-config-heck 0.1.0 → 0.1.4

package/node.js

@@ -148,7 +148,7 @@ const rules = {
     "no-floating-decimal": "error",
     "no-global-assign": "error",
     "no-implicit-coercion": "error",
-    "no-implicit-globals": "error",
+    "no-implicit-globals": "off",
     "no-implied-eval": "error",
     "no-invalid-this": "error",
     "no-iterator": "error",
@@ -244,7 +244,7 @@ const rules = {
             caughtErrors: "all",
         },
     ],
-    "no-use-before-define": "error",
+    "no-use-before-define": ["error", "nofunc"],
 
     // Stylistic issues
     "array-bracket-newline": [
@@ -268,7 +268,7 @@ const rules = {
     "func-call-spacing": "error",
     "func-name-matching": "off",
     "func-names": "off",
-    "func-style": "error",
+    "func-style": ["error", "declaration", { allowArrowFunctions: true }],
     "function-call-argument-newline": ["error", "consistent"],
     "function-paren-newline": ["error", "multiline-arguments"],
     "id-denylist": "off",
@@ -292,22 +292,7 @@ const rules = {
     "keyword-spacing": "error",
     "line-comment-position": "error",
     "linebreak-style": ["error"],
-    "lines-around-comment": [
-        "error",
-        {
-            beforeBlockComment: true,
-            beforeLineComment: true,
-            allowBlockStart: true,
-            allowBlockEnd: false,
-            allowObjectStart: true,
-            allowObjectEnd: false,
-            allowArrayStart: true,
-            allowArrayEnd: false,
-            allowClassStart: true,
-            allowClassEnd: false,
-            applyDefaultIgnorePatterns: true,
-        },
-    ],
+    "lines-around-comment": "off",
     "lines-between-class-members": [
         "error",
         "always",
@@ -351,8 +336,7 @@ const rules = {
     ],
     "no-negated-condition": "off",
 
-    // Improved by unicorn/no-nested-ternary
-    "no-nested-ternary": "off",
+    "no-nested-ternary": "error",
     "no-new-object": "error",
     "no-plusplus": "off",
     "no-restricted-syntax": "off",
@@ -374,14 +358,7 @@ const rules = {
             consistent: true,
         },
     ],
-    "object-curly-spacing": [
-        "error",
-        "always",
-        {
-            arraysInObjects: false,
-            objectsInObjects: false,
-        },
-    ],
+    "object-curly-spacing": ["error", "always"],
     "object-property-newline": [
         "error",
         {
@@ -413,14 +390,7 @@ const rules = {
     ],
     "prefer-exponentiation-operator": "error",
     "prefer-object-spread": "error",
-    "quote-props": [
-        "error",
-        "as-needed",
-        {
-            keywords: true,
-            numbers: true,
-        },
-    ],
+    "quote-props": ["error", "as-needed"],
     quotes: ["error", "double"],
     semi: [
         "error",
@@ -491,8 +461,14 @@ const rules = {
     "prefer-destructuring": [
         "error",
         {
-            object: true,
-            array: false,
+            VariableDeclarator: {
+                array: false,
+                object: true,
+            },
+            AssignmentExpression: {
+                array: false,
+                object: false,
+            },
         },
     ],
     "prefer-numeric-literals": "error",
@@ -525,8 +501,8 @@ const rules = {
     "react/function-component-definition": [
         "error",
         {
-            namedComponents: "function-expression",
-            unnamedComponents: "function-expression",
+            namedComponents: "function-declaration",
+            unnamedComponents: "arrow-function",
         },
     ],
     "react/no-access-state-in-setstate": "error",
@@ -574,7 +550,7 @@ const rules = {
         "error",
         {
             groups: {
-                rendering: ["/^render.+$/", "render"],
+                render: ["render", "/^render.+$/"],
             },
         },
     ],
@@ -634,7 +610,7 @@ const rules = {
     "react/jsx-max-depth": "off",
     "react/jsx-max-props-per-line": "error",
     "react/jsx-newline": "off",
-    "react/jsx-no-bind": "error",
+    "react/jsx-no-bind": ["error", { allowArrowFunctions: true }],
     "react/jsx-no-comment-textnodes": "error",
     "react/jsx-no-constructed-context-values": "error",
     "react/jsx-no-duplicate-props": "error",
@@ -688,7 +664,11 @@ const rules = {
         },
     ],
     "unicorn/consistent-destructuring": "error",
-    "unicorn/consistent-function-scoping": "error",
+    "unicorn/consistent-function-scoping": [
+        "error", {
+            checkArrowFunctions: false,
+        },
+    ],
     "unicorn/custom-error-definition": "off",
     "unicorn/empty-brace-spaces": "off",
     "unicorn/error-message": "error",
@@ -721,9 +701,9 @@ const rules = {
     "unicorn/no-hex-escape": "error",
     "unicorn/no-instanceof-array": "error",
     "unicorn/no-invalid-remove-event-listener": "error",
-    "unicorn/no-keyword-prefix": "error",
+    "unicorn/no-keyword-prefix": "off",
     "unicorn/no-lonely-if": "error",
-    "unicorn/no-nested-ternary": "error",
+    "unicorn/no-nested-ternary": "off",
     "unicorn/no-new-array": "error",
     "unicorn/no-new-buffer": "error",
     "unicorn/no-null": "off",
@@ -763,7 +743,7 @@ const rules = {
     "unicorn/prefer-keyboard-event-key": "error",
     "unicorn/prefer-math-trunc": "error",
     "unicorn/prefer-modern-dom-apis": "error",
-    "unicorn/prefer-module": "error",
+    "unicorn/prefer-module": "off",
     "unicorn/prefer-negative-index": "error",
     "unicorn/prefer-node-protocol": "error",
     "unicorn/prefer-number-properties": "error",
@@ -835,7 +815,7 @@ const overrides = [
             "@typescript-eslint/class-literal-property-style": "error",
             "@typescript-eslint/consistent-indexed-object-style": "error",
             "@typescript-eslint/consistent-type-assertions": "error",
-            "@typescript-eslint/consistent-type-definitions": "off",
+            "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
             "@typescript-eslint/consistent-type-imports": [
                 "error",
                 {
@@ -880,6 +860,12 @@ const overrides = [
                         regex: "^__html$", match: false,
                     },
                 },
+                {
+                    selector: ["parameter"],
+                    modifiers: ["unused"],
+                    format: ["strictCamelCase"],
+                    leadingUnderscore: "require",
+                },
                 {
                     selector: ["typeLike"],
                     format: ["StrictPascalCase"],
@@ -906,7 +892,11 @@ const overrides = [
             "@typescript-eslint/no-for-in-array": "error",
             "@typescript-eslint/no-implicit-any-catch": "error",
             "@typescript-eslint/no-inferrable-types": "error",
-            "@typescript-eslint/no-invalid-void-type": "error",
+            "@typescript-eslint/no-invalid-void-type": [
+                "error", {
+                    allowAsThisParameter: true,
+                },
+            ],
             "@typescript-eslint/no-meaningless-void-operator": "error",
             "@typescript-eslint/no-misused-new": "error",
             "@typescript-eslint/no-misused-promises": "error",
@@ -1079,12 +1069,7 @@ const overrides = [
             "no-useless-constructor": "off",
             "@typescript-eslint/no-useless-constructor": "error",
             "object-curly-spacing": "off",
-            "@typescript-eslint/object-curly-spacing": [
-                "error", "always", {
-                    arraysInObjects: false,
-                    objectsInObjects: false,
-                },
-            ],
+            "@typescript-eslint/object-curly-spacing": ["error", "always"],
             "padding-line-between-statements": "off",
             "@typescript-eslint/padding-line-between-statements": [
                 "error",
@@ -1153,7 +1138,11 @@ const overrides = [
         rules: {
             // eslint-plugin-jest
             "jest/consistent-test-it": "error",
-            "jest/expect-expect": "error",
+            "jest/expect-expect": [
+                "error", {
+                    assertFunctionNames: ["expect*"],
+                },
+            ],
             "jest/max-nested-describe": "off",
             "jest/no-alias-methods": "error",
             "jest/no-commented-out-tests": "error",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-heck",
-  "version": "0.1.0",
+  "version": "0.1.4",
   "description": "Contains an ESLint configuration for ES2015+, TypeScript, and React.",
   "main": "index.js",
   "scripts": {