Only echo command if we are streaming the stdout

github · Oct 10, 2024 · edd7713 · edd7713
1 parent 90c42c4
commit edd7713
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 8 deletions.
diff --git a/lib/actions-util.js b/lib/actions-util.js
diff --git a/lib/actions-util.js.map b/lib/actions-util.js.map
diff --git a/lib/setup-codeql.js b/lib/setup-codeql.js
diff --git a/lib/setup-codeql.js.map b/lib/setup-codeql.js.map
diff --git a/src/actions-util.ts b/src/actions-util.ts
@@ -527,7 +527,9 @@ export async function runTool(
 ): Promise<string> {
   let stdout = "";
   let stderr = "";
-  process.stdout.write(`[command]${cmd} ${args.join(" ")}\n`);
+  if (!opts.noStreamStdout) {
+    process.stdout.write(`[command]${cmd} ${args.join(" ")}\n`);
+  }
   const exitCode = await new toolrunner.ToolRunner(cmd, args, {
     ignoreReturnCode: true,
     listeners: {

diff --git a/src/setup-codeql.ts b/src/setup-codeql.ts
@@ -557,9 +557,9 @@ export const downloadCodeQL = async function (
   );
 
   logger.info(
-    `Added CodeQL bundle to the tool cache (${
-      performance.now() - toolcacheStart
-    } ms).`,
+    `Added CodeQL bundle to the tool cache (${Math.round(
+      performance.now() - toolcacheStart,
+    )} ms).`,
   );
 
   // Defensive check: we expect `cacheDir` to copy the bundle to a new location.