Ignore inconsistent runs that have a status set to 'in_progress' but …

…conclusion is 'success'
softprops · Apr 21, 2023 · 446f151 · 446f151
1 parent ca99add
commit 446f151
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 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", "");