Issue metadata
Sign in to add a comment
|
Run go vet in repo preupload hook. |
||||||||||||||||||||||||
Issue descriptionContext: https://chromium-review.googlesource.com/c/chromiumos/platform/tast-tests/+/1235874/6/src/chromiumos/tast/local/bundles/cros/security/open_fds.go#44 go vet can fail. It will be nice if we can run it in preupload hook, because developers can know the failure quickly. Local experiment: cd ${chromeosroot}/platform/tast-tests GOPATH=${chromeosroot}/chroot/usr/lib/gopath:${chromeosroot}/src/platform/tast:${chromeosroot}/platform/tast-tests go vet chromiumos/tast/local/... worked successfully on my local dev env at least. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by derat@chromium.org
, Sep 26Status: Duplicate (was: Untriaged)