From 9cb8caeb6fe345909bd5aba628f102a1e7d4eb58 Mon Sep 17 00:00:00 2001 From: Ali Ariff Date: Sat, 31 Aug 2024 14:13:35 +0200 Subject: [PATCH] refactor: Update HtmlBeautifier class to use public logChannel --- src/formatter/htmlbeautifier.ts | 2 +- src/formatter/htmlbeautifierProvider.ts | 7 +++++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/formatter/htmlbeautifier.ts b/src/formatter/htmlbeautifier.ts index 33acd52..9acf64d 100644 --- 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 index ee4b67b..54a2527 100644 --- a/src/formatter/htmlbeautifierProvider.ts +++ b/src/formatter/htmlbeautifierProvider.ts @@ -59,14 +59,17 @@ export default class HtmlBeautifierProvider range: vscode.Range ): vscode.ProviderResult { 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 []; } );