Skip to content

Commit 4b9c850

Browse files
fix(ci): require promote-images + deploy-trigger success explicitly for promote-trigger
1 parent 2d908f8 commit 4b9c850

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
@@ -684,9 +684,15 @@ jobs:
684684
promote-trigger:
685685
name: Promote Trigger.dev
686686
needs: [promote-images, deploy-trigger]
687+
# Require both upstreams to have SUCCEEDED explicitly (not just promoted==true):
688+
# a job if without a status-check function keeps the implicit success() gate, but
689+
# spelling it out removes any doubt that a failed promote-images/deploy-trigger
690+
# can't reach this job and promote tasks with no app retag/cutover.
687691
if: >-
688692
github.event_name == 'push' &&
689693
(github.ref == 'refs/heads/main' || github.ref == 'refs/heads/staging') &&
694+
needs.promote-images.result == 'success' &&
695+
needs.deploy-trigger.result == 'success' &&
690696
needs.promote-images.outputs.promoted == 'true'
691697
runs-on: blacksmith-4vcpu-ubuntu-2404
692698
# Must exceed the poll script's OVERALL_TIMEOUT (70 min, covering a prod deploy

0 commit comments

Comments
 (0)