From e1e560a141a3923b090f78fb712f57cc0a5b6e69 Mon Sep 17 00:00:00 2001 From: Felix Lange Date: Sat, 6 Oct 2018 12:40:42 +0200 Subject: [PATCH] build: don't run vet before tests Recent versions of Go run vet during 'go test' and we have a dedicated lint job. --- build/ci.go | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/build/ci.go b/build/ci.go index c5a107e1d5..1e3f31124b 100644 --- a/build/ci.go +++ b/build/ci.go @@ -332,13 +332,10 @@ func doTest(cmdline []string) { } packages = build.ExpandPackagesNoVendor(packages) - // Run analysis tools before the tests. - build.MustRun(goTool("vet", packages...)) - // Run the actual tests. - gotest := goTool("test", buildFlags(env)...) // Test a single package at a time. CI builders are slow // and some tests run into timeouts under load. + gotest := goTool("test", buildFlags(env)...) gotest.Args = append(gotest.Args, "-p", "1") if *coverage { gotest.Args = append(gotest.Args, "-covermode=atomic", "-cover")