ci: fix job dependency in test workflow

The main build matrix does not currently depend on the start job. Hence the build matrix starts in parallel with the start job (the latter is responsible for notifying the start of a build). Which means the start message can get delayed. Fix that. Change-Id: Ia7710fd3762ab8656df8cd82dcaee198a7e91c83 Reviewed-on: https://cue-review.googlesource.com/c/cue/+/8401 Reviewed-by: CUE cueckoo <cueckoo@gmail.com> Reviewed-by: Marcel van Lohuizen <mpvl@golang.org>
cuelang · Jan 29, 2021 · 34333ce · 34333ce
1 parent 3088513
commit 34333ce
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -23,6 +23,7 @@ jobs:
         github.run_id }}"}'' -b ~/.gitcookies https://cue-review.googlesource.com/a/changes/$(basename
         $(dirname $GITHUB_REF))/revisions/$(basename $GITHUB_REF)/review'
   test:
+    needs: start
     strategy:
       fail-fast: false
       matrix:

diff --git a/internal/ci/workflows.cue b/internal/ci/workflows.cue
@@ -50,6 +50,7 @@ test: _#bashWorkflow & {
 			]
 		}
 		test: {
+			needs:     "start"
 			strategy:  _#testStrategy
 			"runs-on": "${{ matrix.os }}"
 			steps: [