@spscommerce/eslint-config-typescript 0.0.7 → 0.0.8

package/lib/rules/bestPractices.js

@@ -237,9 +237,12 @@ exports.bestPractices = {
     "@typescript-eslint/no-magic-numbers": [
         "error",
         {
-            ignore: [0,
+            ignore: [
+                -1,
+                0,
                 1,
-                2],
+                2,
+            ],
             ignoreArrayIndexes: true,
         },
     ],

package/lib/rules/possibleErrors.js

@@ -93,8 +93,9 @@ exports.possibleErrors = {
     /**
      * disallow double-negation boolean casts in a boolean context ✅ 🔧
      * https://eslint.org/docs/rules/no-extra-boolean-cast
+     * Disabled because it conflicts with @typescript-eslint/strict-boolean-expressions
      */
-    "no-extra-boolean-cast": "error",
+    "no-extra-boolean-cast": "off",
     /**
      * disallow unnecessary parentheses 🔧
      * https://eslint.org/docs/rules/no-extra-parens

package/lib/rules/typescript.js

@@ -111,10 +111,7 @@ exports.typescript = {
      * Requires expressions of type void to appear in statement position 🔧 💭
      * https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-confusing-void-expression.md
      */
-    "@typescript-eslint/no-confusing-void-expression": [
-        "error",
-        { "ignoreArrowShorthand": true }
-    ],
+    "@typescript-eslint/no-confusing-void-expression": ["error", { ignoreArrowShorthand: true }],
     /**
      * Disallow the delete operator with computed key expressions 🔧
      * https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-dynamic-delete.md
@@ -284,8 +281,10 @@ exports.typescript = {
     /**
      * Prefers a non-null assertion over explicit type cast when possible 🔧 💭
      * https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/non-nullable-type-assertion-style.md
+     * Disabled because it enforces use of postfix ! operator, which is banned by
+     * @typescript-eslint/no-non-null-assertion (a recommended rule)
      */
-    "@typescript-eslint/non-nullable-type-assertion-style": "error",
+    "@typescript-eslint/non-nullable-type-assertion-style": "off",
     /**
      * Prefer usage of as const over literal type ✅ 🔧
      * https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/prefer-as-const.md
@@ -433,5 +432,5 @@ exports.typescript = {
      * one by using a union or an optional/rest parameter
      * https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/unified-signatures.md
      */
-    "@typescript-eslint/unified-signatures": "error",
+    "@typescript-eslint/unified-signatures": "warn",
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spscommerce/eslint-config-typescript",
-  "version": "0.0.7",
+  "version": "0.0.8",
   "description": "SPS official linter configuration for TypeScript.",
   "main": "lib/index.js",
   "scripts": {