golangci-lint should be run for all build tags, not just for Linux

[jandubois]

Right now only files that are compiled on Linux are checked.

There is currently a macOS specific warning that is not caught by CI:

golangci-lint run ./...
pkg/vz/vz_driver_darwin.go:206:2: directive `//nolint:revive // error-strings` is unused for linter "revive" (nolintlint)
	//nolint:revive // error-strings
	^
1 issues:
* nolintlint: 1

I don't know if we need to run the linter on each platform separately, or if we can simply add something like --build-tags=linux,windows,darwin,386,amd64,arm,arm64 to the commandline (well, this doesn't work; not sure why though).

[nirs]

So the issue is not checking code that is not compiled on the current system/arch?

[jandubois]

So the issue is not checking code that is not compiled on the current system/arch?

Yes

[afbjorklund]

Could you do it with GOOS, or does it need to be run on the other platforms?
(I would expect it to work, unless there is some native C code or something)
EDIT: Of course, for the Vz driver that is exactly what is needed - so not possible

Just thinking it would be nice to avoid the more expensive runners, if possible