Merge pull request #5367 from dotty-staging/fix/tracing-zip

Don't zip workspace if tracing is disabled

Author: smarter

vscode-dotty/src/tracer.ts

@@ -75,15 +75,17 @@ export class Tracer {
 
     run(): vscode.OutputChannel | undefined {
         const consentCommandDisposable = vscode.commands.registerCommand(consentCommandName, () => this.askForTracingConsent())
-        if (this.isTracingEnabled && this.tracingConsent.get() === 'no-answer') this.askForTracingConsent()
-        this.initializeAsyncWorkspaceDump()
-
-        const lspOutputChannel = this.createLspOutputChannel()
-        const statusBarItem = this.createStatusBarItem()
-        for (const disposable of [consentCommandDisposable, lspOutputChannel, statusBarItem]) {
-            if (disposable) this.ctx.extensionContext.subscriptions.push(disposable)
+        if (this.isTracingEnabled) {
+            if (this.tracingConsent.get() === 'no-answer') this.askForTracingConsent()
+            this.initializeAsyncWorkspaceDump()
+
+            const lspOutputChannel = this.createLspOutputChannel()
+            const statusBarItem = this.createStatusBarItem()
+            for (const disposable of [consentCommandDisposable, lspOutputChannel, statusBarItem]) {
+                if (disposable) this.ctx.extensionContext.subscriptions.push(disposable)
+            }
+            return lspOutputChannel
         }
-        return lspOutputChannel
     }
 
     private askForTracingConsent(): void {