mirror of
https://github.com/tailscale/tailscale.git
synced 2026-03-25 17:53:03 -04:00
.github/workflows: limit vet to the tailscale.com module (#19084)
This repo's module is tailscale.com, and the tailscale-client-go-v2 repo uses tailscale.com/client/tailscale/v2. It seems from #19010 that if we have the client module as a dependency in this module, go vet will start to consider the client module as part of tailscale.com/... I'm not sure if this is a bug in go vet, but for now let's take the easy fix and specify ./... instead. In my testing, it seems like this is sufficient to make sure it just walks the file hierarchy and doesn't find the client module as a sub-path. Updates tailscale/corp#38418 Signed-off-by: Tom Proctor <tomhjp@users.noreply.github.com>
This commit is contained in:
6
.github/workflows/vet.yml
vendored
6
.github/workflows/vet.yml
vendored
@@ -36,4 +36,8 @@ jobs:
|
||||
|
||||
- name: Run 'go vet'
|
||||
working-directory: src
|
||||
run: ./tool/go vet -vettool=/tmp/vettool tailscale.com/...
|
||||
# Must use ./... instead of tailscale.com/... because the latter will
|
||||
# include the v2 go client (tailscale.com/client/tailscale/v2) if it's
|
||||
# a dependency in our go.mod file. Possibly a go vet bug, but avoid
|
||||
# cross-repo vetting for now so we can safely add the dependency.
|
||||
run: ./tool/go vet -vettool=/tmp/vettool ./...
|
||||
|
||||
Reference in New Issue
Block a user