workflows/pr-comment-builds: remove always() condition

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
This commit is contained in:
Thilo Fromm 2023-05-30 18:13:11 +02:00
parent af686c0f41
commit 5e1a2c270d

View File

@ -57,7 +57,7 @@ jobs:
update_sdk: update_sdk:
needs: check_maintainer_membership needs: check_maintainer_membership
if: ( always() && needs.check_maintainer_membership.result == 'success' if: ( needs.check_maintainer_membership.result == 'success'
&& contains(github.event.comment.body, '/update-sdk') ) && contains(github.event.comment.body, '/update-sdk') )
name: "Build an updated SDK container" name: "Build an updated SDK container"
# SDK build needs access to bincache ssh secret # SDK build needs access to bincache ssh secret
@ -66,7 +66,7 @@ jobs:
build_image: build_image:
needs: [ check_maintainer_membership, update_sdk ] needs: [ check_maintainer_membership, update_sdk ]
if: ( always() && needs.check_maintainer_membership.result == 'success' if: ( needs.check_maintainer_membership.result == 'success'
&& ( contains(github.event.comment.body, '/build-image') || needs.update_sdk.result == 'success' ) ) && ( contains(github.event.comment.body, '/build-image') || needs.update_sdk.result == 'success' ) )
name: "Build the OS image" name: "Build the OS image"
uses: ./.github/workflows/ci.yaml uses: ./.github/workflows/ci.yaml