Merge pull request #257 from crazy-max/list-targets-check

list-targets: check targets are set
docker · Oct 8, 2024 · d38ec1b · d38ec1b
2 parents 51e939b + 782be98
commit d38ec1b
Showing 1 changed file with 27 additions and 9 deletions.
diff --git a/.github/workflows/ci-subaction.yml b/.github/workflows/ci-subaction.yml
@@ -38,9 +38,15 @@ jobs:
         with:
           workdir: ./test/group
       -
-        name: Show targets
-        run: |
-          echo targets=${{ steps.gen.outputs.targets }}
+        name: Check targets
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const targets = `${{ steps.gen.outputs.targets }}`;
+            if (!targets) {
+              core.setFailed('No targets generated');
+            }
+            core.info(`targets=${targets}`);
 
   list-targets-group-matrix:
     runs-on: ubuntu-latest
@@ -56,9 +62,15 @@ jobs:
           workdir: ./test/group-matrix
           target: validate
       -
-        name: Show targets
-        run: |
-          echo targets=${{ steps.gen.outputs.targets }}
+        name: Check targets
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const targets = `${{ steps.gen.outputs.targets }}`;
+            if (!targets) {
+              core.setFailed('No targets generated');
+            }
+            core.info(`targets=${targets}`);
 
   list-targets-multi-files:
     runs-on: ubuntu-latest
@@ -76,6 +88,12 @@ jobs:
             docker-bake.json
             docker-bake.hcl
       -
-        name: Show targets
-        run: |
-          echo targets=${{ steps.gen.outputs.targets }}
+        name: Check targets
+        uses: actions/github-script@v7
+        with:
+          script: |
+            const targets = `${{ steps.gen.outputs.targets }}`;
+            if (!targets) {
+              core.setFailed('No targets generated');
+            }
+            core.info(`targets=${targets}`);