Merge pull request #541 from Tradeshift/POPS-3158_disable_tag_with_pr…

…efix fix: disable tag with prefix
Tradeshift · Jan 17, 2024 · 59a9d42 · 59a9d42
2 parents 82c5efe + 58fedce
commit 59a9d42
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