ESLint fixes

Author: JamesMessinger

src/action/index.ts

@@ -24,7 +24,6 @@ async function main(): Promise<void> {
     // Puglish to NPM
     let results = await npmPublish(options);
 
-    // tslint:disable: no-console
     if (results.type === "none") {
       console.log(`\n📦 ${results.package} v${results.version} is already published to NPM`);
     }
@@ -62,5 +61,5 @@ function debugHandler(message: string, data?: object) {
   debug(message);
 }
 
-// tslint:disable-next-line: no-floating-promises
+// eslint-disable-next-line @typescript-eslint/no-floating-promises
 main();

src/cli/index.ts

@@ -1,4 +1,3 @@
-// tslint:disable: no-console
 import { join } from "path";
 import { npmPublish } from "../npm-publish";
 import { readManifest } from "../read-manifest";
@@ -35,7 +34,6 @@ export async function main(args: string[]): Promise<void> {
       let results = await npmPublish(options);
 
       if (!options.quiet) {
-        // tslint:disable: no-console
         if (results.type === "none") {
           console.log(`\n📦 ${results.package} v${results.version} is already published to NPM`);
         }

src/cli/parse-args.ts

@@ -1,4 +1,3 @@
-// tslint:disable: no-console
 import * as commandLineArgs from "command-line-args";
 import { Options } from "../options";
 import { ExitCode } from "./exit-code";

src/index.ts

@@ -1,4 +1,3 @@
-// tslint:disable: no-default-export
 import { npmPublish } from "./npm-publish";
 
 // Exprot the external type definitions as named exports
@@ -11,5 +10,5 @@ export default npmPublish;
 
 // CommonJS default export hack
 if (typeof module === "object" && typeof module.exports === "object") {
-  module.exports = Object.assign(module.exports.default, module.exports);  // tslint:disable-line: no-unsafe-any
+  module.exports = Object.assign(module.exports.default, module.exports);
 }

src/npm-config.ts

@@ -51,13 +51,13 @@ function updateConfig(config: string, { registry, debug }: NormalizedOptions): s
  */
 async function getNpmConfigPath({ debug }: NormalizedOptions): Promise<string> {
   try {
-    debug(`Running command: npm config get userconfig`);
+    debug("Running command: npm config get userconfig");
 
     let process = await ezSpawn.async("npm", "config", "get", "userconfig");
     return process.stdout.trim();
   }
   catch (error) {
-    throw ono(error, `Unable to determine the NPM config file path.`);
+    throw ono(error, "Unable to determine the NPM config file path.");
   }
 }
 
@@ -76,7 +76,7 @@ async function readNpmConfig(configPath: string, { debug }: NormalizedOptions):
   }
   catch (error) {
     if ((error as NodeJS.ErrnoException).code === "ENOENT") {
-      debug(`OLD NPM CONFIG: <none>`);
+      debug("OLD NPM CONFIG: <none>");
       return "";
     }
 

src/npm.ts

@@ -78,6 +78,7 @@ function getNpmEnvironment(options: NormalizedOptions): NodeJS.ProcessEnv | unde
   let needsToken = Boolean(options.token && process.env.INPUT_TOKEN !== options.token);
 
   if (needsToken) {
+    // eslint-disable-next-line @typescript-eslint/naming-convention
     return { ...process.env, INPUT_TOKEN: options.token };
   }
 }

src/read-manifest.ts

@@ -32,7 +32,7 @@ export async function readManifest(path: string, debug?: Debug): Promise<Manifes
     let { name, version } = JSON.parse(json) as Record<string, unknown>;
 
     if (typeof name !== "string" || name.trim().length === 0) {
-      throw new TypeError(`Invalid package name`);
+      throw new TypeError("Invalid package name");
     }
 
     let manifest: Manifest = {