Skip to content

Commit

Permalink
CI: use debian-based images, only test -race on amd64
Browse files Browse the repository at this point in the history
debian-based images have gcc (needed for -race) and git
(possible with alpine image but would need more install steps)

go test -race is not supported on 386

Also, go test without -v, but list tests first:
go-diskqueue tests produce a lot of distracting error logging output,
and if a test fails, a pretty good amount of logging context is provided.
  • Loading branch information
ploxiln committed Sep 12, 2021
1 parent a14d574 commit 2e64e45
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
- "amd64"
- "386"

container: "golang:${{matrix.gover}}-alpine"
container: "golang:${{matrix.gover}}"
env:
GOARCH: "${{matrix.goarch}}"

Expand All @@ -30,8 +30,12 @@ jobs:

- name: test
run: |
GOMAXPROCS=1 go test -v
GOMAXPROCS=4 go test -v -race
go test -list Test
GOMAXPROCS=1 go test
if [ "$GOARCH" = "amd64" ]; then
echo "With -race:"
GOMAXPROCS=4 go test -race
fi
- name: lint
run: |
Expand Down

0 comments on commit 2e64e45

Please sign in to comment.