-
Notifications
You must be signed in to change notification settings - Fork 18k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmd/vet: documentation incomplete / outdated #7298
Labels
Comments
... to clarify the usage message a bit. Labels changed: added documentation. Owner changed to @adg. Status changed to Accepted. |
Did you notice this message? > To run the vet tool with specific options, run 'go tool vet'. If you follow the instruction, here's what happens: $ go tool vet Usage of vet: vet [flags] directory... vet [flags] files... # Must be a single package -all=true: check everything; disabled if any explicit check is requested -asmdecl=false: check assembly against Go declarations -assign=false: check for useless assignments -atomic=false: check for common mistaken usages of the sync/atomic package -buildtags=false: check that +build tags are valid -composites=false: check that composite literals used field-keyed elements -compositewhitelist=true: use composite white list; for testing only -copylocks=false: check that locks are not passed by value -methods=false: check that canonically named methods are canonically defined -nilfunc=false: check for comparisons between functions and nil -printf=false: check printf-like invocations -printfuncs="": comma-separated list of print function names to check -rangeloops=false: check that range loop variables are used correctly -shadow=false: check for shadowed variables (experimental; must be set explicitly) -shadowstrict=false: whether to be strict about shadowing; can be noisy -structtags=false: check that struct field tags have canonical format -test=false: for testing only: sets -all and -shadow -unreachable=false: check for unreachable code -v=false: verbose Isn't that what you were looking for? Status changed to WaitingForReply. |
Yes, that's it. I guess the message didn't really clue me in to it. The distinction between running "go vet" and "go tool vet" is subtle enough that I must have glossed over it, but I don't see any easy way that can be improved short of increasing my reading comprehension :) The part that says > For more about vet, see 'godoc code.google.com/p/go.tools/cmd/vet'. still leads to outdated documentation though. |
Wow, yeah, that's pretty out of date. Sent a CL: https://golang.org/cl/61480044 Status changed to Started. |
This issue was closed by revision golang/tools@47596b4. Status changed to Fixed. |
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: