eslint-config-isaacscript 3.1.2 → 3.3.0

package/base.js

@@ -357,7 +357,18 @@ module.exports = {
 
     /**
      * Documentation:
-     * https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/prefer-default-export.md
+     * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md
+     *
+     * Not defined in the parent configs.
+     *
+     * The case against default exports is layed out here:
+     * https://basarat.gitbook.io/typescript/main-1/defaultisbad
+     */
+    "import/no-default-export": "warn",
+
+    /**
+     * Documentation:
+     * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/prefer-default-export.md
      *
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/imports.js
@@ -446,6 +457,16 @@ module.exports = {
       },
     ],
 
+    /**
+     * Documentation:
+     * https://eslint.org/docs/latest/rules/no-unused-expressions
+     *
+     * Not defined in parent configs.
+     *
+     * An unused expression which has no effect on the state of the program indicates a logic error.
+     */
+    "no-unused-expressions": "warn",
+
     /**
      * Documentation:
      * https://eslint.org/docs/latest/rules/prefer-destructuring
@@ -453,7 +474,7 @@ module.exports = {
      * Defined at:
      * https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/es6.js
      *
-     * - Array destructuring can result in non-intuitive code.
+     * Array destructuring can result in non-intuitive code.
      */
     "prefer-destructuring": [
       "warn",

package/jsdoc.js

@@ -176,21 +176,20 @@ module.exports = {
 
     // - jsdoc/require-param-type - Not needed in TypeScript.
 
-    // - jsdoc/require-param - It's overboard for every function to document every parameter.
-    // TODO
-    /*
     "jsdoc/require-param": [
       "warn",
       {
+        // We only activate the rule when there are one or more parameters.
+        // https://github.com/gajus/eslint-plugin-jsdoc/issues/920
         contexts: [
           {
             context: "FunctionDeclaration",
-            comment: 'JsdocBlock > JsdocTag[tag="param"]',
+            comment: 'JsdocBlock:has(JsdocTag[tag="param"])',
           },
         ],
       },
     ],
-    */
+
     // - jsdoc/require-property - Probably not needed in TypeScript.
 
     /**

package/mod.js

@@ -131,6 +131,16 @@ module.exports = {
      */
     "isaacscript/no-invalid-default-map": "warn",
 
+    /**
+     * Documentation:
+     * https://github.com/IsaacScript/isaacscript/blob/main/packages/eslint-plugin-isaacscript/docs/rules/no-throw.md
+     *
+     * Not defined in the parent configs.
+     *
+     * The `throw` keyword should never be used in Isaac mods.
+     */
+    "isaacscript/no-throw": "warn",
+
     /**
      * Documentation:
      * https://eslint.org/docs/latest/rules/no-bitwise

package/package.json

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