@open-turo/eslint-config-typescript 15.0.0-pr-374.337.1.1 → 15.0.0

package/index.js

@@ -103,8 +103,20 @@ module.exports = {
         prefer: "type-imports",
       },
     ],
+    /** We do not need to force people to wrap `void`-return implicit arrow returns with braces just for a lint rule. TypeScript alone covers functionality, by the return type being `void`. */
+    "@typescript-eslint/no-confusing-void-expression": "off",
+    /** Included as part of `strict-type-checked`, but nothing we want to enforce. */
+    "@typescript-eslint/no-deprecated": "off",
     /** Prefers `import type {}` syntax over `import { type }` if all imports are type-only */
     "@typescript-eslint/no-import-type-side-effects": "error",
+    /** A relatively stylistic rule, downgraded to "off" to limit breaking changes in the update that includes `strict-type-checked`. */
+    "@typescript-eslint/no-unnecessary-boolean-literal-compare": "off",
+    /** This rule can help us identify syntax that is more defensive than the types suggest. Unfortunately, it may be protecting us from runtime errors where the type definitions are incorrect. As such, it is turned "off", as even "warn" auto-fixes source code. */
+    "@typescript-eslint/no-unnecessary-condition": "off",
+    /** There is readability benefit to passing in type arguments that match defaults. If defaults change, we may prefer the manual inspection of all the types changed, too. */
+    "@typescript-eslint/no-unnecessary-type-arguments": "off",
+    /** Errors on generic type parameters that are only used once, even though that helps with return type inference. */
+    "@typescript-eslint/no-unnecessary-type-parameters": "off",
     "@typescript-eslint/no-unused-vars": [
       "error",
       {

package/package.json

@@ -2,8 +2,8 @@
   "author": "Turo engineering",
   "description": "Turo eslint configuration for typescript",
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "8.21.0",
-    "@typescript-eslint/parser": "8.21.0",
+    "@typescript-eslint/eslint-plugin": "8.22.0",
+    "@typescript-eslint/parser": "8.22.0",
     "eslint-config-prettier": "10.0.1",
     "eslint-import-resolver-typescript": "3.7.0",
     "eslint-plugin-import": "2.31.0",
@@ -42,5 +42,5 @@
     "access": "public"
   },
   "repository": "https://github.com/open-turo/eslint-config-typescript",
-  "version": "15.0.0-pr-374.337.1.1"
+  "version": "15.0.0"
 }

package/test/__snapshots__/test.spec.js.snap

@@ -704,10 +704,10 @@ exports[`validate config load config file in ESLint to validate all rules are co
       "error",
     ],
     "@typescript-eslint/no-confusing-void-expression": [
-      "error",
+      "off",
     ],
     "@typescript-eslint/no-deprecated": [
-      "error",
+      "off",
     ],
     "@typescript-eslint/no-duplicate-enum-values": [
       "error",
@@ -788,16 +788,16 @@ exports[`validate config load config file in ESLint to validate all rules are co
       "error",
     ],
     "@typescript-eslint/no-unnecessary-boolean-literal-compare": [
-      "error",
+      "off",
     ],
     "@typescript-eslint/no-unnecessary-condition": [
-      "error",
+      "off",
     ],
     "@typescript-eslint/no-unnecessary-template-expression": [
       "error",
     ],
     "@typescript-eslint/no-unnecessary-type-arguments": [
-      "error",
+      "off",
     ],
     "@typescript-eslint/no-unnecessary-type-assertion": [
       "error",
@@ -806,7 +806,7 @@ exports[`validate config load config file in ESLint to validate all rules are co
       "error",
     ],
     "@typescript-eslint/no-unnecessary-type-parameters": [
-      "error",
+      "off",
     ],
     "@typescript-eslint/no-unsafe-argument": [
       "error",