refactor: Update HtmlBeautifier class to use public logChannel

aliariff · Aug 31, 2024 · 9cb8cae · 9cb8cae
1 parent 7115d2f
commit 9cb8cae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/formatter/htmlbeautifier.ts b/src/formatter/htmlbeautifier.ts
@@ -3,7 +3,7 @@ import * as cp from "child_process";
 const isWsl = require("is-wsl");
 
 export default class HtmlBeautifier {
-  private logChannel: vscode.LogOutputChannel;
+  public logChannel: vscode.LogOutputChannel;
 
   constructor() {
     this.logChannel = vscode.window.createOutputChannel("ERB Beautifier", {

diff --git a/src/formatter/htmlbeautifierProvider.ts b/src/formatter/htmlbeautifierProvider.ts
@@ -59,14 +59,17 @@ export default class HtmlBeautifierProvider
     range: vscode.Range
   ): vscode.ProviderResult<vscode.TextEdit[]> {
     if (this.shouldIgnore(document)) {
-      console.log(`Ignoring ${document.fileName}`);
+      this.htmlbeautifier.logChannel.info(`Ignoring ${document.fileName}`);
       return [];
     }
 
     return this.htmlbeautifier.format(document.getText(range)).then(
       (result) => [new vscode.TextEdit(range, result)],
       (err) => {
-        console.error(`Error formatting ${document.fileName}:`, err);
+        this.htmlbeautifier.logChannel.error(
+          `Error formatting ${document.fileName}:`,
+          err
+        );
         return [];
       }
     );