eslint-config-isaacscript 3.6.1 → 3.7.1

package/base-eslint.js

@@ -112,8 +112,7 @@ module.exports = {
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/style.js
      *
-     * - We move the selector for "for..of" loops, since they are commonly used.
-     * - We add a selector for "empty" invocations of the "array.push()" method.
+     * - We remove the `ForOfStatement` selector, since they are commonly used.
      */
     "no-restricted-syntax": [
       "warn",
@@ -132,11 +131,6 @@ module.exports = {
         message:
           "`with` is disallowed in strict mode because it makes code impossible to predict and optimize.",
       },
-      {
-        selector:
-          "CallExpression[callee.property.name='push'][arguments.length=0]",
-        message: "push must always be called with at least one argument.",
-      },
     ],
 
     /**

package/base-jsdoc.js

@@ -65,10 +65,14 @@ module.exports = {
           "noSelf",
           "noSelfInFile",
 
-          // Ignore tags used in ts-json-schema-generator:
+          // Ignore tags used in `ts-json-schema-generator`:
           // https://github.com/vega/ts-json-schema-generator
           "minimum",
           "maximum",
+
+          // Ignore tags used in `eslint-plugin-isaacscript`:
+          // https://github.com/IsaacScript/isaacscript/blob/main/packages/eslint-plugin-isaacscript/docs/rules/require-variadic-function-argument.md
+          "allowEmptyVariadic",
         ],
       },
     ],

package/base-typescript-eslint.js

@@ -140,7 +140,7 @@ module.exports = {
      * Defined at:
      * https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/strict.ts
      *
-     * This can modify the type of boolean declarations to "boolean | undefined", which is undesired
+     * This can modify the type of boolean declarations to `boolean | undefined`, which is undesired
      * in some circumstances:
      * https://github.com/typescript-eslint/typescript-eslint/issues/5269
      */

package/base.js

@@ -342,7 +342,14 @@ module.exports = {
      */
     "unicorn/prefer-node-protocol": "warn",
 
-    // TEST
+    /**
+     * Documentation:
+     * https://eslint.org/docs/latest/rules/no-restricted-syntax
+     *
+     * Not defined in parent configs.
+     *
+     * Prevent superfluous type annotations, which can cause bugs with widened types.
+     */
     // https://github.com/typescript-eslint/typescript-eslint/issues/6446
     "no-restricted-syntax": [
       "warn",

package/mod.js

@@ -142,6 +142,16 @@ module.exports = {
      */
     "isaacscript/no-throw": "warn",
 
+    /**
+     * Documentation:
+     * https://github.com/IsaacScript/isaacscript/blob/main/packages/eslint-plugin-isaacscript/docs/rules/require-v-registration.md
+     *
+     * Not defined in the parent configs.
+     *
+     * We must explicitly enable this rule, since it should only apply to IsaacScript mods.
+     */
+    "isaacscript/require-v-registration": "warn",
+
     /**
      * Documentation:
      * https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/file-extension-in-import.md

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-isaacscript",
-  "version": "3.6.1",
+  "version": "3.7.1",
   "description": "A sharable ESLint config for TypeScript and IsaacScript projects.",
   "keywords": [
     "eslint",