eslint-config-beslogic 2.4.6 → 2.4.7

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.4.7
+
+- No longer recommending to explicitely annotate void returning awaitable arrow functions as `(param): void => { statement }` in JavaScript.
+
 ## 2.4.6
 
 - Replaced `@typescript-eslint/*` with `typescript-eslint`

package/javascript.js

@@ -31,12 +31,6 @@ const noRestrictedSyntax = [
     "message":
       "Prefer `Promise.resolve().then()` instead of `setTimeout` without a specified delay. " +
       "See: https://developer.mozilla.org/en-US/docs/Web/API/setTimeout#reasons_for_delays_longer_than_specified and https://angular.io/guide/change-detection-zone-pollution"
-  },
-  {
-    "selector":
-      "ArrowFunctionExpression[body.body.length=1][body.body.0.type=\"ExpressionStatement\"]:not([returnType.typeAnnotation.type=\"TSVoidKeyword\"])",
-    "message": "Consider removing braces around the arrow function's body. " +
-      "If it really needs to return `void` you can mark its return type explicitely with `(param): void => { statement }` to disable this warning."
   }
 ]
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-beslogic",
-  "version": "2.4.6",
+  "version": "2.4.7",
   "description": "ESLint rules, plugins and configs used at Beslogic",
   "main": "index.js",
   "// dependencies": "Run for ourselves when we install dependencies w/o running npm i afterward",

package/typescript.js

@@ -12,6 +12,12 @@ const hasTotalFunctions = !!resolveModuleLocation("eslint-plugin-total-functions
 const commaDangleDefault = "always-multiline"
 const noRestrictedSyntax = [
   ...javascriptConfig.rules?.["no-restricted-syntax"] ?? {},
+  {
+    "selector":
+      "ArrowFunctionExpression[body.body.length=1][body.body.0.type=\"ExpressionStatement\"]:not([returnType.typeAnnotation.type=\"TSVoidKeyword\"])",
+    "message": "Consider removing braces around the arrow function's body. " +
+      "If it really needs to return `void` you can mark its return type explicitely with `(param): void => { statement }` to disable this warning."
+  },
   // @typescript-eslint won't make this a rule
   // https://github.com/typescript-eslint/typescript-eslint/issues/4247
   {