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.

Sign up for GitHub

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

Jump to bottom

go vet need add to integration testing #564

Closed

xiaods opened this issue Oct 13, 2016 · 0 comments

Contributor

xiaods commented Oct 13, 2016 •

edited

Loading

Actual Behavior:

What is the issue? *

found go vet reports many bugs. it should be notice to developer.

Testing Example

$ go vet ./src/utils/config/.../
src/utils/config/config.go:15: struct field tag `env:"CRANE_DOCKER_TLS_VERIFY" envDefault:false,required:"true"` not compatible with reflect.StructTag.Get: bad syntax for struct tag value
src/utils/config/config.go:53: no formatting directive in Fatalf call
exit status 1

The text was updated successfully, but these errors were encountered:

xiaods closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment