Merge pull request #542 from Tradeshift/POPS-3158_add_support_on_sche…

…duled_workflows fix: add tag with prefix support on schedule GHA
Tradeshift · Jan 17, 2024 · ba83ffb · ba83ffb
2 parents 75bb8c0 + 895af75
commit ba83ffb
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 6 deletions.
diff --git a/action.yml b/action.yml
@@ -75,7 +75,6 @@ inputs:
       based on the type of build: on master or on pull rquest.
       An image built from a pull request is tagged with pr-<number>-<SHA>.
       An image built on merge to master branch is tagged with master-<SHA>.
-    default: "true"
   useqemu:
     required: false
     description: >

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/docker.ts b/src/docker.ts
@@ -117,8 +117,12 @@ async function getSHATagWithPrefix(repository: string): Promise<string> {
 
   let shaWithPrefix: string;
   const ref = event?.ref || '';
-  if (ref !== '' && ref === `refs/heads/${default_branch}`) {
-    // on merge to default branch
+  const schedule = event?.schedule || '';
+  if (
+    (ref !== '' && ref === `refs/heads/${default_branch}`) ||
+    schedule !== ''
+  ) {
+    // on merge to default branch or on scheduled workflow
     shaWithPrefix = `${default_branch}-${sha}`;
   } else {
     // on pull request