From 871d7520edbb95811adb7ea82a6eceef9553f062 Mon Sep 17 00:00:00 2001 From: Will Dollman Date: Tue, 24 Jun 2025 13:35:55 +0100 Subject: [PATCH 1/2] Add a check that only succeeds if all matrix tests pass This is required so that we can make the checks mandatory in branch protection - otherwise the names will keep changing --- .github/workflows/go-ci.yml | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/.github/workflows/go-ci.yml b/.github/workflows/go-ci.yml index 5ab699bfde..c1962feeee 100644 --- a/.github/workflows/go-ci.yml +++ b/.github/workflows/go-ci.yml @@ -30,3 +30,18 @@ jobs: - run: | go test -race -v ./... go test -v ./... + + # This job provides a stable name for branch protection rules + # It will only pass if all Go CI matrix tests above pass + go-test-complete: + runs-on: ubuntu-latest + needs: go-test + if: always() + steps: + - name: Check Go CI matrix job results + run: | + if [[ "${{ needs.go-test.result }}" != "success" ]]; then + echo "One or more Go CI tests failed" + exit 1 + fi + echo "All Go CI tests passed" From 747a4b37b3654e34c0e9c346550a36af2fc12454 Mon Sep 17 00:00:00 2001 From: Will Dollman Date: Tue, 24 Jun 2025 13:37:09 +0100 Subject: [PATCH 2/2] Tweak name --- .github/workflows/go-ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/go-ci.yml b/.github/workflows/go-ci.yml index c1962feeee..33dce1f948 100644 --- a/.github/workflows/go-ci.yml +++ b/.github/workflows/go-ci.yml @@ -38,7 +38,7 @@ jobs: needs: go-test if: always() steps: - - name: Check Go CI matrix job results + - name: Check Go CI results run: | if [[ "${{ needs.go-test.result }}" != "success" ]]; then echo "One or more Go CI tests failed"