Update eslint-plugin-promise to v7.2.1

- Enable `promise/prefer-catch` as a warning.

package.json

@@ -50,7 +50,7 @@
         "eslint-plugin-jsdoc": "^50.6.0",
         "eslint-plugin-n": "^17.14.0",
         "eslint-plugin-prefer-arrow": "^1.2.3",
-        "eslint-plugin-promise": "^7.1.0",
+        "eslint-plugin-promise": "^7.2.1",
         "eslint-plugin-rxjs": "^5.0.3",
         "eslint-plugin-security": "^3.0.1",
         "eslint-plugin-simple-import-sort": "^12.1.1",
@@ -98,7 +98,7 @@
         "eslint-plugin-jsdoc": "50.6.0",
         "eslint-plugin-n": "17.14.0",
         "eslint-plugin-prefer-arrow": "1.2.3",
-        "eslint-plugin-promise": "7.1.0",
+        "eslint-plugin-promise": "7.2.1",
         "eslint-plugin-rxjs": "5.0.3",
         "eslint-plugin-security": "3.0.1",
         "eslint-plugin-simple-import-sort": "12.1.1",

src/rules/promise/index.adoc

@@ -70,15 +70,11 @@ to enforce best practices for JavaScript promises.
 | Error
 
 
-| `link:{eslint-promise-rules}/spec-only.md[promise/spec-only]`
-|
-|
-| Error
-
-| `link:{eslint-promise-rules}/valid-params.md[promise/valid-params]`
+| `link:{eslint-promise-rules}/prefer-await-to-callbacks.md[promise/prefer-await-to-callbacks]`
 |
 |
-| Error
+| Off
+4+| Requires ES2017 `async`/`await`.
 
 | `link:{eslint-promise-rules}/prefer-await-to-then.md[promise/prefer-await-to-then]`
 |
@@ -88,10 +84,19 @@ to enforce best practices for JavaScript promises.
 and would be turned off either way,
 as Promises should be preferred.
 
-| `link:{eslint-promise-rules}/prefer-await-to-callbacks.md[promise/prefer-await-to-callbacks]`
+| `link:{eslint-promise-rules}/prefer-catch.md[promise/prefer-catch]`
 |
 |
-| Off
-4+| Requires ES2017 `async`/`await`.
+| Warning
+
+| `link:{eslint-promise-rules}/spec-only.md[promise/spec-only]`
+|
+|
+| Error
+
+| `link:{eslint-promise-rules}/valid-params.md[promise/valid-params]`
+|
+|
+| Error
 
 |===

src/rules/promise/index.ts

@@ -6,18 +6,29 @@ export const promiseConfig = {
         promise: eslintPluginPromise,
     },
     rules: {
-        'promise/catch-or-return': 'error',
+        'promise/catch-or-return': ['error', {
+            allowThen: false,
+            allowThenStrict: false,
+            allowFinally: true,
+        }],
         'promise/no-return-wrap': 'error',
         'promise/param-names': 'warn',
-        'promise/always-return': 'error',
+        'promise/always-return': ['error', {
+            ignoreAssignmentVariable: [
+                'globalThis',
+            ],
+        }],
         'promise/no-multiple-resolved': 'error',
         'promise/no-native': 'off',
         'promise/no-nesting': 'error',
         'promise/no-promise-in-callback': 'error',
-        'promise/no-callback-in-promise': 'error',
+        'promise/no-callback-in-promise': ['error', {
+            timeoutsErr: false,
+        }],
         'promise/avoid-new': 'off',
         'promise/no-new-statics': 'warn',
         'promise/no-return-in-finally': 'error',
+        'promise/prefer-catch': 'warn',
         'promise/spec-only': ['error', {
             allowedMethods: [],
         }],