Set job status to success if analyze succeeded

github · Jan 24, 2024 · 0af1cd9 · 0af1cd9
1 parent 42cc9a3
commit 0af1cd9
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 5 deletions.
diff --git a/lib/init-action-post-helper.js b/lib/init-action-post-helper.js
diff --git a/lib/init-action-post-helper.js.map b/lib/init-action-post-helper.js.map
diff --git a/lib/init-action-post.js b/lib/init-action-post.js
diff --git a/src/init-action-post-helper.ts b/src/init-action-post-helper.ts
@@ -148,6 +148,10 @@ export async function tryUploadSarifIfRunFailed(
       return createFailedUploadFailedSarifResult(e);
     }
   } else {
+    core.exportVariable(
+      EnvVar.JOB_STATUS,
+      process.env[EnvVar.JOB_STATUS] ?? JobStatus.Success,
+    );
     return {
       upload_failed_run_skipped_because:
         "Analyze Action completed successfully",

diff --git a/src/init-action-post.ts b/src/init-action-post.ts
@@ -87,11 +87,11 @@ async function runWrapper() {
     ...statusReportBase,
     ...uploadFailedSarifResult,
     job_status:
-      (process.env[EnvVar.JOB_STATUS] as JobStatus) ?? JobStatus.Success,
+      (process.env[EnvVar.JOB_STATUS] as JobStatus) ?? JobStatus.Unknown,
   };
   core.info(
     `Sending job_status field as ${
-      (process.env[EnvVar.JOB_STATUS] as JobStatus) ?? JobStatus.Success
+      (process.env[EnvVar.JOB_STATUS] as JobStatus) ?? JobStatus.Unknown
     }`,
   );
   await sendStatusReport(statusReport);