Skip to content

Commit a12e378

Browse files
committed
Fix Docker workflow dependencies by explicitly requiring test-dev-image
- Update `needs` to include both `build-dev-image` and `test-dev-image` for better dependency management - Adjust conditional expressions to use `needs.<job>.result` instead of `jobs.<job>.result` Signed-off-by: Phillip Sitbon <phillip.sitbon@gmail.com>
1 parent d385a11 commit a12e378

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/docker-publish.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -100,8 +100,8 @@ jobs:
100100
name: Push Dev Images
101101
runs-on: ubuntu-latest
102102

103-
needs: build-dev-image
104-
if: ${{ always() && needs.build-dev-image.result == 'success' && (jobs.test-dev-image.result == 'success' || jobs.test-dev-image.result == 'skipped') }}
103+
needs: [build-dev-image, test-dev-image]
104+
if: ${{ always() && needs.build-dev-image.result == 'success' && (needs.test-dev-image.result == 'success' || needs.test-dev-image.result == 'skipped') }}
105105

106106
permissions:
107107
contents: read
@@ -174,8 +174,8 @@ jobs:
174174
name: Build and Push Image
175175
runs-on: ubuntu-latest
176176

177-
needs: build-dev-image
178-
if: ${{ always() && needs.build-dev-image.result == 'success' && (jobs.test-dev-image.result == 'success' || jobs.test-dev-image.result == 'skipped') }}
177+
needs: [build-dev-image, test-dev-image]
178+
if: ${{ always() && needs.build-dev-image.result == 'success' && (needs.test-dev-image.result == 'success' || needs.test-dev-image.result == 'skipped') }}
179179

180180
permissions:
181181
contents: read

0 commit comments

Comments
 (0)