fix: ignore inconsistent runs that have a status set to 'in_progress'…

… but conclusion is 'success' (#59) * Ignore inconsistent runs that have a status set to 'in_progress' but conclusion is 'success' * Bump action node version to 16 * Upgrade dependencies, run npm dedupe * run fmt and build Signed-off-by: Rui Chen <[email protected]> --------- Signed-off-by: Rui Chen <[email protected]> Co-authored-by: gustaff-weldon <[email protected]>
softprops · Jun 19, 2024 · df7e268 · df7e268
1 parent 63921e5
commit df7e268
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/main.ts b/src/main.ts
@@ -8,9 +8,8 @@ async function run() {
   try {
     const input = parseInput(env);
     debug(
-      `Parsed inputs (w/o token): ${(({ githubToken, ...inputs }) => inputs)(
-        input,
-      )}`,
+      `Parsed inputs (w/o token): ${(({ githubToken, ...inputs }) =>
+        JSON.stringify(inputs))(input)}`,
     );
     const github = new OctokitGitHub(input.githubToken);
     debug(`Fetching workflows for ${input.owner}/${input.repo}...`);

diff --git a/src/wait.ts b/src/wait.ts
@@ -57,6 +57,17 @@ export class Waiter implements Wait {
     this.debug(`Found ${runs.length} ${this.workflowId} runs`);
     const previousRuns = runs
       .filter((run) => run.id < this.input.runId)
+      .filter((run) => {
+        const isSuccessful: boolean = run.conclusion === "success";
+
+        if (isSuccessful) {
+          this.debug(
+            `Skipping run ${run.id}, status: ${run.status}, conclusion: ${run.conclusion}`,
+          );
+        }
+
+        return !isSuccessful;
+      })
       .sort((a, b) => b.id - a.id);
     if (!previousRuns || !previousRuns.length) {
       setOutput("force_continued", "");