Merge pull request #239 from MichaelDeBoey/patch-4

chore: update `eslint-plugin-eslint-plugin` to latest
eslint-community · Dec 18, 2021 · 3f5520f · 3f5520f
2 parents ea6536b + 8dd426a
commit 3f5520f
Show file tree

Hide file tree

Showing 16 changed files with 6,610 additions and 10 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -27,7 +27,7 @@
     "doctoc": "^2.1.0",
     "eslint": "^7.32.0",
     "eslint-config-prettier": "^8.3.0",
-    "eslint-plugin-eslint-plugin": "^3.6.1",
+    "eslint-plugin-eslint-plugin": "^4.1.0",
     "eslint-plugin-jest": "^25.3.0",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^3.4.1",

diff --git a/rules/always-return.js b/rules/always-return.js
@@ -62,6 +62,7 @@ module.exports = {
     docs: {
       url: getDocsUrl('always-return'),
     },
+    schema: [],
   },
   create(context) {
     // funcInfoStack is a stack representing the stack of currently executing

diff --git a/rules/avoid-new.js b/rules/avoid-new.js
@@ -13,6 +13,7 @@ module.exports = {
     docs: {
       url: getDocsUrl('avoid-new'),
     },
+    schema: [],
   },
   create(context) {
     return {

diff --git a/rules/catch-or-return.js b/rules/catch-or-return.js
@@ -18,6 +18,31 @@ module.exports = {
     messages: {
       terminationMethod: 'Expected {{ terminationMethod }}() or return',
     },
+    schema: [
+      {
+        type: 'object',
+        properties: {
+          allowFinally: {
+            type: 'boolean',
+          },
+          allowThen: {
+            type: 'boolean',
+          },
+          terminationMethod: {
+            oneOf: [
+              { type: 'string' },
+              {
+                type: 'array',
+                items: {
+                  type: 'string',
+                },
+              },
+            ],
+          },
+        },
+        additionalProperties: false,
+      },
+    ],
   },
   create(context) {
     const options = context.options[0] || {}

diff --git a/rules/no-callback-in-promise.js b/rules/no-callback-in-promise.js
@@ -19,6 +19,20 @@ module.exports = {
     messages: {
       callback: 'Avoid calling back inside of a promise.',
     },
+    schema: [
+      {
+        type: 'object',
+        properties: {
+          exceptions: {
+            type: 'array',
+            items: {
+              type: 'string',
+            },
+          },
+        },
+        additionalProperties: false,
+      },
+    ],
   },
   create(context) {
     return {

diff --git a/rules/no-native.js b/rules/no-native.js
@@ -32,6 +32,7 @@ module.exports = {
     messages: {
       name: '"{{name}}" is not defined.',
     },
+    schema: [],
   },
   create(context) {
     /**

diff --git a/rules/no-nesting.js b/rules/no-nesting.js
@@ -15,6 +15,7 @@ module.exports = {
     docs: {
       url: getDocsUrl('no-nesting'),
     },
+    schema: [],
   },
   create(context) {
     return {

diff --git a/rules/no-new-statics.js b/rules/no-new-statics.js
@@ -10,6 +10,7 @@ module.exports = {
       url: getDocsUrl('no-new-statics'),
     },
     fixable: 'code',
+    schema: [],
   },
   create(context) {
     return {

diff --git a/rules/no-promise-in-callback.js b/rules/no-promise-in-callback.js
@@ -15,6 +15,7 @@ module.exports = {
     docs: {
       url: getDocsUrl('no-promise-in-callback'),
     },
+    schema: [],
   },
   create(context) {
     return {

diff --git a/rules/no-return-in-finally.js b/rules/no-return-in-finally.js
@@ -9,6 +9,7 @@ module.exports = {
     docs: {
       url: getDocsUrl('no-return-in-finally'),
     },
+    schema: [],
   },
   create(context) {
     return {

diff --git a/rules/no-return-wrap.js b/rules/no-return-wrap.js
@@ -45,6 +45,17 @@ module.exports = {
       resolve: 'Avoid wrapping return values in Promise.resolve',
       reject: 'Expected throw instead of Promise.reject',
     },
+    schema: [
+      {
+        type: 'object',
+        properties: {
+          allowReject: {
+            type: 'boolean',
+          },
+        },
+        additionalProperties: false,
+      },
+    ],
   },
   create(context) {
     const options = context.options[0] || {}

diff --git a/rules/param-names.js b/rules/param-names.js
@@ -8,6 +8,7 @@ module.exports = {
     docs: {
       url: getDocsUrl('param-names'),
     },
+    schema: [],
   },
   create(context) {
     return {

diff --git a/rules/prefer-await-to-callbacks.js b/rules/prefer-await-to-callbacks.js
@@ -11,6 +11,7 @@ module.exports = {
     messages: {
       error: 'Avoid callbacks. Prefer Async/Await.',
     },
+    schema: [],
   },
   create(context) {
     function checkLastParamsForCallback(node) {

diff --git a/rules/prefer-await-to-then.js b/rules/prefer-await-to-then.js
@@ -13,6 +13,7 @@ module.exports = {
     docs: {
       url: getDocsUrl('prefer-await-to-then'),
     },
+    schema: [],
   },
   create(context) {
     /** Returns true if node is inside yield or await expression. */

diff --git a/rules/valid-params.js b/rules/valid-params.js
@@ -11,6 +11,7 @@ module.exports = {
         'Ensures the proper number of arguments are passed to Promise functions',
       url: getDocsUrl('valid-params'),
     },
+    schema: [],
   },
   create(context) {
     return {