Skip to content

Commit 81d73da

Browse files
fix(ci): require promote-images + deploy-trigger success explicitly for promote-trigger
1 parent 67d8224 commit 81d73da

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -802,9 +802,15 @@ jobs:
802802
promote-trigger:
803803
name: Promote Trigger.dev
804804
needs: [promote-images, deploy-trigger]
805+
# Require both upstreams to have SUCCEEDED explicitly (not just promoted==true):
806+
# a job if without a status-check function keeps the implicit success() gate, but
807+
# spelling it out removes any doubt that a failed promote-images/deploy-trigger
808+
# can't reach this job and promote tasks with no app retag/cutover.
805809
if: >-
806810
github.event_name == 'push' &&
807811
(github.ref == 'refs/heads/main' || github.ref == 'refs/heads/staging') &&
812+
needs.promote-images.result == 'success' &&
813+
needs.deploy-trigger.result == 'success' &&
808814
needs.promote-images.outputs.promoted == 'true'
809815
runs-on: blacksmith-4vcpu-ubuntu-2404
810816
# Must exceed the poll script's OVERALL_TIMEOUT (70 min, covering a prod deploy

0 commit comments

Comments
 (0)