Remove some logging in 0.7.1, simplify some code

chalet-org · Feb 11, 2024 · 77b859e · 77b859e
1 parent b5d096f
commit 77b859e
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 20 deletions.
diff --git a/Extension/CHANGELOG.md b/Extension/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Change Log
 
+## Version 0.7.2
+
+- Remove some logging in 0.7.1, simplify some code
+
 ## Version 0.7.1
 
 - When ending the Chalet process via the trash icon on unix platforms, use `tree-kill` to make sure child processes close as well

diff --git a/Extension/package.json b/Extension/package.json
@@ -2,7 +2,7 @@
 	"name": "vscode-chalet",
 	"displayName": "Chalet",
 	"description": "Support for Chalet projects in Visual Studio Code",
-	"version": "0.7.1",
+	"version": "0.7.2",
 	"publisher": "chalet-org",
 	"license": "BSD-3-Clause",
 	"icon": "chalet-logo.png",

diff --git a/Extension/src/Terminal/TerminalProcess.ts b/Extension/src/Terminal/TerminalProcess.ts
@@ -74,26 +74,13 @@ class TerminalProcess {
             const pid = this.subprocess.pid;
 
             const callback = (err: any) => {
-                // Note: We don't care about the error
-                // if (!!err) OutputChannel.logError(err);
-
-                if (this.platform === VSCodePlatform.Windows) {
-                    if (!!err) {
-                        console.error(err);
-                        console.error("there was an error halting the process");
-                    } else {
-                        this.subprocess = null;
-                        onHalt?.();
-                    }
+                if (!!err) {
+                    // OutputChannel.logError(err);
+                    console.error(err);
+                    console.error("there was an error halting the process");
                 } else {
-                    if (!!err) {
-                        console.error(err);
-                        console.error("there was an error halting the process");
-                    } else {
-                        console.log("task killed successfully");
-                        this.subprocess = null;
-                        onHalt?.();
-                    }
+                    this.subprocess = null;
+                    onHalt?.();
                 }
             };