Skip to content

Commit dce7de3

Browse files
committed
ci: install all packages first
All packages need to be installed before we can run (some) of the checks and code generators reliably. More precisely, anything that using x/tools/go/loader is fragile (this includes stringer, vet and others). The blocking issue is golang/go#14120; see golang/go#10249 for some more concrete discussion on `stringer` and golang/go#16086 for `vet`.
1 parent 3d017fb commit dce7de3

File tree

1 file changed

+9
-1
lines changed

1 file changed

+9
-1
lines changed

build/teamcity-check.sh

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,15 @@
11
#!/usr/bin/env bash
22
set -exuo pipefail
3-
build/builder.sh make check 2>&1 | go-test-teamcity
3+
# All packages need to be installed before we can run (some) of the checks
4+
# and code generators reliably. More precisely, anything that using
5+
# x/tools/go/loader is fragile (this includes stringer, vet and others).
6+
#
7+
# The blocking issue is https://github.com/golang/go/issues/14120; see
8+
# https://github.com/golang/go/issues/10249 for some more concrete discussion
9+
# on `stringer` and https://github.com/golang/go/issues/16086 for `vet`.
410
build/builder.sh make gotestdashi 2>&1
11+
12+
build/builder.sh make check 2>&1 | go-test-teamcity
513
build/builder.sh go generate ./... 2>&1
614
build/builder.sh /bin/bash -c '! git status --porcelain | read || (git status; git diff -a; exit 1)' 2>&1
715

0 commit comments

Comments
 (0)