Disable vet as part of go test

[silverwind]

go vet implicitely runs as part of go test, but we already have make lint-go-vet, so we were essentially running it twice. This should hopefully make go test slightly faster.

Ref: https://pkg.go.dev/cmd/go#hdr-Test_packages

As part of building a test binary, go test runs go vet

[silverwind]

Actually I see we ran vet only with -vettool which replaces the default checks:

The -vettool=prog flag selects a different analysis tool with alternative
or additional checks.

So I've added a command that now runs plain vet as part of lint-go-vet.

[silverwind]

Need to check whether gitea-vit replaces or extends the builtin checks. If the latter, we only need to run go vet once.

[techknowlogick]

On a lower powered machine this definitely helps.

[silverwind]

Yeah I agree, go test in general is a massively slow test runner compared to others like vitest, I guess it's partly because it runs vet. But let's wait on the investigation above until I'm certain whether one or two vet calls are needed.