Merge pull request #48 from WXYC/bugfix/workflow-type-error

I understand now... I think
WXYC · Feb 21, 2025 · 900e91d · 900e91d
2 parents f4d4593 + bd0733a
commit 900e91d
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/.github/workflows/deploy-auto.yml b/.github/workflows/deploy-auto.yml
@@ -9,4 +9,4 @@ jobs:
   trigger-build-and-deploy:
     uses: ./.github/workflows/deploy-base.yml
     with:
-      dry-run: 'false'
+      dry-run: false
diff --git a/.github/workflows/deploy-base.yml b/.github/workflows/deploy-base.yml
@@ -5,11 +5,12 @@ on:
     inputs:
       dry_run:
         required: false
-        default: 'true'
+        type: boolean
+        default: true
 
 jobs:
   dry-run:
-    if: github.event.inputs.dry_run == 'true'
+    if: ${{ inputs.dry_run }}
     runs-on: ubuntu-latest
 
     steps:
@@ -79,7 +80,7 @@ jobs:
             docker ps || exit 1
 
   build-and-deploy:
-    if: github.event.inputs.dry_run != 'true'
+    if: ${{ !inputs.dry_run }}
     runs-on: ubuntu-latest
 
     steps:

diff --git a/.github/workflows/deploy-manual.yml b/.github/workflows/deploy-manual.yml
@@ -6,10 +6,11 @@ on:
       dry_run:
         description: 'Perform a dry run'
         required: false
-        default: 'true'
+        type: boolean
+        default: true
 
 jobs:
   trigger-build-and-deploy:
     uses: ./.github/workflows/deploy-base.yml
     with:
-      dry-run: github.events.inputs.dry_run
+      dry-run: ${{ inputs.dry_run }}