From 993a1dcb6f9129c1fd64fb0b7320a12b41b1905d Mon Sep 17 00:00:00 2001 From: Malte Viering Date: Mon, 11 May 2026 07:37:44 +0000 Subject: [PATCH 1/2] ci: run test jobs in parallel --- .github/workflows/test.yaml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml index 18eba2604..466eec960 100644 --- a/.github/workflows/test.yaml +++ b/.github/workflows/test.yaml @@ -25,8 +25,6 @@ jobs: run: go test -v ./... test-with-docker: - # We don't need to run this longer test if the previous one already failed. - needs: test-without-docker runs-on: ubuntu-latest services: dind: From 82278bfdb0fcd33ae3f051300c6365c707bae42e Mon Sep 17 00:00:00 2001 From: Malte Viering Date: Wed, 13 May 2026 06:30:47 +0000 Subject: [PATCH 2/2] ci: consolidate into single test job covering ./... Remove redundant test-without-docker job and expand the Docker-based test job to cover the entire repository (./...) instead of only ./internal/..., as suggested in PR review. --- .github/workflows/test.yaml | 18 +++--------------- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml index 466eec960..402951a55 100644 --- a/.github/workflows/test.yaml +++ b/.github/workflows/test.yaml @@ -12,19 +12,7 @@ on: - '*' jobs: - test-without-docker: - runs-on: ubuntu-latest - steps: - - name: Checkout code - uses: actions/checkout@v6 - - name: Set up Go - uses: actions/setup-go@v6 - with: - go-version-file: 'go.mod' - - name: Test quickly without Docker - run: go test -v ./... - - test-with-docker: + test: runs-on: ubuntu-latest services: dind: @@ -38,7 +26,7 @@ jobs: uses: actions/setup-go@v6 with: go-version-file: 'go.mod' - - name: Run tests with Docker and calculate coverage + - name: Run tests and calculate coverage run: | export GITHUB_ACTIONS=1 export POSTGRES_CONTAINER=1 @@ -46,7 +34,7 @@ jobs: echo "Running tests..." go test -v \ -coverpkg=./internal/... \ - -coverprofile=profile.cov ./internal/... + -coverprofile=profile.cov ./... # Exclude e2e files from the coverage profile so they don't # count against the coverage percentage.