- Add some debug lines to see which part failed in CI

- Unified return for the dependencies command to return boolean

Author: michael-weng

src/commands/dependencies/resolve.ts

@@ -24,7 +24,8 @@ import { executeTaskWithUI, updateAfterError } from "../utilities";
 export async function resolveDependencies(ctx: WorkspaceContext) {
     const current = ctx.currentFolder;
     if (!current) {
-        return;
+        ctx.outputChannel.log("currentFolder is not set.");
+        return false;
     }
     return await resolveFolderDependencies(current);
 }

src/commands/dependencies/unedit.ts

@@ -26,7 +26,8 @@ import { FolderContext } from "../../FolderContext";
 export async function uneditDependency(identifier: string, ctx: WorkspaceContext) {
     const currentFolder = ctx.currentFolder;
     if (!currentFolder) {
-        return;
+        ctx.outputChannel.log("currentFolder is not set.");
+        return false;
     }
     ctx.outputChannel.log(`unedit dependency ${identifier}`, currentFolder.name);
     const status = `Reverting edited dependency ${identifier} (${currentFolder.name})`;
@@ -80,12 +81,13 @@ async function uneditFolderDependency(
 
             if (result === "No") {
                 ctx.outputChannel.log(execError.stderr, folder.name);
-                return;
+                return false;
             }
             await uneditFolderDependency(folder, identifier, ctx, ["--force"]);
         } else {
             ctx.outputChannel.log(execError.stderr, folder.name);
             vscode.window.showErrorMessage(`${execError.stderr}`);
         }
+        return false;
     }
 }

src/commands/dependencies/update.ts

@@ -24,7 +24,8 @@ import { executeTaskWithUI, updateAfterError } from "./../utilities";
 export async function updateDependencies(ctx: WorkspaceContext) {
     const current = ctx.currentFolder;
     if (!current) {
-        return;
+        ctx.outputChannel.log("currentFolder is not set.");
+        return false;
     }
     return await updateFolderDependencies(current);
 }

src/commands/dependencies/useLocal.ts

@@ -30,6 +30,7 @@ export async function useLocalDependency(
 ): Promise<boolean> {
     const currentFolder = ctx.currentFolder;
     if (!currentFolder) {
+        ctx.outputChannel.log("currentFolder is not set.");
         return false;
     }
     const folders = await vscode.window.showOpenDialog({

src/ui/SwiftOutputChannel.ts

@@ -29,7 +29,7 @@ export class SwiftOutputChannel implements vscode.OutputChannel {
         logStoreLinesSize: number = 250_000 // default to capturing 250k log lines
     ) {
         this.name = name;
-        this.logToConsole = process.env["CI"] !== "1" && logToConsole;
+        //        this.logToConsole = process.env["CI"] !== "1" && logToConsole;
         this.channel = vscode.window.createOutputChannel(name);
         this.logStore = new RollingLog(logStoreLinesSize);
     }