Merge pull request #91 from stuartleeks/sl/fix-push-filter-check

Fix push filter check
devcontainers · Jul 8, 2021 · 23289aa · 23289aa
2 parents 3684470 + e5d1002
commit 23289aa
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 6 deletions.
diff --git a/github-action/dist/index.js b/github-action/dist/index.js
diff --git a/github-action/dist/index.js.map b/github-action/dist/index.js.map
diff --git a/github-action/lib/main.js b/github-action/lib/main.js
@@ -92,7 +92,7 @@ function runPost() {
                 return;
             }
         }
-        if (pushOption !== 'always') {
+        else if (pushOption !== 'always') {
             core.setFailed(`Unexpected push value ('${pushOption})'`);
             return;
         }

diff --git a/github-action/src/main.ts b/github-action/src/main.ts
@@ -86,9 +86,7 @@ async function runPost(): Promise<void> {
 			)
 			return
 		}
-	}
-
-	if (pushOption !== 'always') {
+	} else if (pushOption !== 'always') {
 		core.setFailed(`Unexpected push value ('${pushOption})'`)
 		return
 	}