Add more info messages to user, rename log printing function

github · Aug 2, 2022 · a758ec5 · a758ec5
1 parent eeee462
commit a758ec5
Show file tree

Hide file tree

Showing 9 changed files with 22 additions and 13 deletions.
diff --git a/lib/analyze-action-post.js b/lib/analyze-action-post.js
diff --git a/lib/analyze-action-post.js.map b/lib/analyze-action-post.js.map
diff --git a/lib/debug-artifacts.js b/lib/debug-artifacts.js
diff --git a/lib/debug-artifacts.js.map b/lib/debug-artifacts.js.map
diff --git a/lib/init-action-post.js b/lib/init-action-post.js
diff --git a/lib/init-action-post.js.map b/lib/init-action-post.js.map
diff --git a/src/analyze-action-post.ts b/src/analyze-action-post.ts
@@ -22,6 +22,9 @@ async function run(uploadSarifDebugArtifact: Function) {
 
   // Upload Actions SARIF artifacts for debugging
   if (config?.debugMode) {
+    core.info(
+      "Debug mode is on. Uploading available SARIF files as Actions debugging artifact..."
+    );
     const outputDir = actionsUtil.getRequiredInput("output");
     await uploadSarifDebugArtifact(config, outputDir);
   }

diff --git a/src/debug-artifacts.ts b/src/debug-artifacts.ts
@@ -69,7 +69,7 @@ export async function uploadSarifDebugArtifact(
   await uploadDebugArtifacts(toUpload, outputDir, config.debugArtifactName);
 }
 
-export async function uploadFinalLogsDebugArtifact(config: Config) {
+export async function printDebugLogs(config: Config) {
   core.info("Debug mode is on. Printing CodeQL debug logs...");
   for (const language of config.languages) {
     const databaseDirectory = getCodeQLDatabasePath(config, language);

diff --git a/src/init-action-post.ts b/src/init-action-post.ts
@@ -14,7 +14,7 @@ import { getActionsLogger } from "./logging";
 async function run(
   uploadDatabaseBundleDebugArtifact: Function,
   uploadLogsDebugArtifact: Function,
-  uploadFinalLogsDebugArtifact: Function
+  printDebugLogs: Function
 ) {
   const logger = getActionsLogger();
 
@@ -27,9 +27,13 @@ async function run(
 
   // Upload appropriate Actions artifacts for debugging
   if (config?.debugMode) {
+    core.info(
+      "Debug mode is on. Uploading available database bundles and logs as Actions debugging artifacts..."
+    );
     await uploadDatabaseBundleDebugArtifact(config, logger);
     await uploadLogsDebugArtifact(config);
-    await uploadFinalLogsDebugArtifact(config);
+
+    await printDebugLogs(config);
   }
 }
 
@@ -38,7 +42,7 @@ async function runWrapper() {
     await run(
       debugArtifacts.uploadDatabaseBundleDebugArtifact,
       debugArtifacts.uploadLogsDebugArtifact,
-      debugArtifacts.uploadFinalLogsDebugArtifact
+      debugArtifacts.printDebugLogs
     );
   } catch (error) {
     core.setFailed(`init action cleanup failed: ${error}`);