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>
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 8c72bc8..ebc1b05 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -23,6 +23,7 @@
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
index 78c9502..288e9bd 100644
--- a/internal/ci/workflows.cue
+++ b/internal/ci/workflows.cue
@@ -50,6 +50,7 @@
]
}
test: {
+ needs: "start"
strategy: _#testStrategy
"runs-on": "${{ matrix.os }}"
steps: [
