Disables rule @typescript-eslint/no-non-null-assertion

- Cleans-up code on `help.ts` that was affected by the rule.
GoogleChromeLabs · Jan 30, 2020 · 7f2784c · 7f2784c · fcsan88 · Apr 24, 2022
1 parent e339453
commit 7f2784c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -43,7 +43,10 @@
       "extends": [
         "plugin:@typescript-eslint/eslint-recommended",
         "plugin:@typescript-eslint/recommended"
-      ]
+      ],
+      "rules": {
+        "@typescript-eslint/no-non-null-assertion": 0
+      }
     }
   ]
 }
diff --git a/src/cli/cmds/help.ts b/src/cli/cmds/help.ts
@@ -59,12 +59,7 @@ export async function help(args: ParsedArgs, log = new Log('help')): Promise<voi
   const command = args._[1];
   const message = HELP_MESSAGES.get(command) || HELP_MESSAGES.get('main');
 
-  // Should never happen, as we know we have a message for main,
-  // but TypeScript doesn't know it.
-  if (message === undefined) {
-    log.error(`Could not find help message for args: ${args}`);
-    return;
-  }
-  log.info(message);
+  // We know we have a message for 'main', in case the command is invalid.
+  log.info(message!);
   return;
 }