eslint-config-isaacscript 4.5.0 → 4.7.0

package/base-eslint.js

@@ -561,28 +561,8 @@ const SUGGESTIONS = {
   /** Superseded by the `no-autofix/prefer-const` rule (since the autofix is usually unwanted). */
   "prefer-const": "off",
 
-  /**
-   * Object destructuring is enforced but array destructuring is not. This matches usage in the
-   * general TypeScript ecosystem.
-   */
-  "prefer-destructuring": [
-    "error",
-    {
-      VariableDeclarator: {
-        array: false,
-        object: true,
-      },
-      AssignmentExpression: {
-        array: false,
-        object: true,
-      },
-    },
-    {
-      // We disable this for renamed properties, this this is a valid use-case.
-      // e.g. `const collectibleUsedToShowFlight = CollectibleType.FATE;`
-      enforceForRenamedProperties: false,
-    },
-  ],
+  /** Superseded by the `@typescript-eslint/prefer-destructuring` rule. */
+  "prefer-destructuring": "off",
 
   "prefer-exponentiation-operator": "error",
 
@@ -644,8 +624,16 @@ const SUGGESTIONS = {
   /**
    * Partially superseded by `isaacscript/format-jsdoc-comments` and
    * `isaacscript/format-line-comments`, but those rules do not handle trailing line comments.
+   *
+   * The `markers` option is provided to make this rule ignore lines that start with "///".
    */
-  "spaced-comment": "error",
+  "spaced-comment": [
+    "error",
+    "always",
+    {
+      markers: ["/"],
+    },
+  ],
 
   /** The `never` option is provided to make the rule stricter. */
   strict: ["error", "never"],

package/base-typescript-eslint.js

@@ -138,6 +138,30 @@ const SUPPORTED_RULES = {
   "@typescript-eslint/non-nullable-type-assertion-style": "error",
   "@typescript-eslint/parameter-properties": "error",
   "@typescript-eslint/prefer-as-const": "error",
+
+  /**
+   * Object destructuring is enforced but array destructuring is not. This matches usage in the
+   * general TypeScript ecosystem.
+   */
+  "@typescript-eslint/prefer-destructuring": [
+    "error",
+    {
+      VariableDeclarator: {
+        array: false,
+        object: true,
+      },
+      AssignmentExpression: {
+        array: false,
+        object: true,
+      },
+    },
+    {
+      // We disable this for renamed properties, this this is a valid use-case.
+      // e.g. `const collectibleUsedToShowFlight = CollectibleType.FATE;`
+      enforceForRenamedProperties: false,
+    },
+  ],
+
   "@typescript-eslint/prefer-enum-initializers": "error",
   "@typescript-eslint/prefer-for-of": "error",
   "@typescript-eslint/prefer-function-type": "error",

package/package.json

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