Remove TODO comments from non-redundant console.debug suppressions

Author: dduugg

src/configuration.ts

@@ -15,7 +15,6 @@ export const getConfig: () => PackwerkConfig = () => {
   const conf = vs.workspace.getConfiguration('ruby.packwerk');
   let executable = conf.get('executable', 'bin/packwerk check');
 
-  // TODO: likely redundant debug logging; consider removing
   // eslint-disable-next-line no-console
   console.debug(`[DEBUG] Parsing config, found executable '${executable}'`);
 

src/outputParser.ts

@@ -10,7 +10,6 @@ export function parseOutput(str: string): PackwerkOutput {
 
     let arr: RegExpExecArray;
     while ((arr = regex.exec(str)) !== null) {
-      // TODO: likely redundant debug logging; consider removing
       // eslint-disable-next-line no-console
       console.log('[DEBUG] Parsed regular expression', arr);
       const file = arr[1];

src/packwerk.ts

@@ -48,15 +48,13 @@ export class Packwerk {
     let relativeFileName = fileName.replace(currentPath + '/', '');
 
     let onDidExec = (error: Error, stdout: string, stderr: string) => {
-      // TODO: likely redundant debug logging; consider removing
       // eslint-disable-next-line no-console
       console.debug('[DEBUG] Finished running command, in onDidExec');
       // eslint-disable-next-line no-console
       console.debug('[DEBUG] Error, stderr', error, stderr);
       this.reportError(error, stderr);
       let packwerk = this.parse(stdout);
       if (packwerk === undefined || packwerk === null) {
-        // TODO: likely redundant debug logging; consider removing
         // eslint-disable-next-line no-console
         console.debug('[DEBUG] packwerk is undefined or null, returning from onDidExec');
         return;
@@ -81,7 +79,6 @@ export class Packwerk {
             /[\u001b\u009b][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]/g, '');
 
           const message = decolorizedMessage;
-          // TODO: likely redundant debug logging; consider removing
           // eslint-disable-next-line no-console
           console.debug('[DEBUG] Adding vscode.Diagnostic:', { range, message });
           const diagnostic = new vscode.Diagnostic(
@@ -138,7 +135,6 @@ export class Packwerk {
     cb: (err: Error, stdout: string, stderr: string) => void
   ): cp.ChildProcess {
     let command = `${this.config.executable} ${fileName}`;
-    // TODO: likely redundant debug logging; consider removing
     // eslint-disable-next-line no-console
     console.debug(`[DEBUG] Running command ${command}`);
 
@@ -151,7 +147,6 @@ export class Packwerk {
   private parse(output: string): PackwerkOutput | null {
     let packwerk: PackwerkOutput;
     if (output.length < 1) {
-      // TODO: likely redundant debug logging; consider removing
       // eslint-disable-next-line no-console
       console.debug(`[DEBUG] Output is ${output}`);
       let message = `command ${this.config.executable} returns empty output! please check configuration.`;

src/taskQueue.ts

@@ -96,7 +96,6 @@ export class TaskQueue {
     let uriString = uri.toString(true);
     this.tasks.forEach((task) => {
       if (task.uri.toString(true) === uriString) {
-        // TODO: likely redundant debug logging; consider removing
         // eslint-disable-next-line no-console
         console.debug(`[DEBUG] Canceling existing task for ${task.uri}`);
         task.cancel();