gofmt84%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command

• go-flagz/dynjson.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• go-flagz/dynstringset_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• go-flagz/examples/server/http.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• go-flagz/examples/server_kube/http.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• go-flagz/configmap/updater.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!

gocyclo100%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

No problems detected. Good job!

golint81%

Golint is a linter for Go source code.

• go-flagz/protobuf/dynproto3.go
  • Line 40: warning: comment on exported type DynProto3Value should be of the form "DynProto3Value ..." (with optional leading article) (golint)
  • Line 140: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• go-flagz/dynjson.go
  • Line 126: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• go-flagz/fileread_flag.go
  • Line 52: warning: exported method FileReadValue.Set should have comment or be unexported (golint)
  • Line 57: warning: exported method FileReadValue.Type should have comment or be unexported (golint)

• go-flagz/monitoring/collector.go
  • Line 17: warning: exported const DynamicFlags should have comment (or a comment on this block) or be unexported (golint)

• go-flagz/configmap/updater.go
  • Line 36: warning: exported type Updater should have comment or be unexported (golint)
  • Line 46: warning: exported function New should have comment or be unexported (golint)
  • Line 59: warning: exported method Updater.Initialize should have comment or be unexported (golint)
  • Line 79: warning: comment on exported method Updater.Stop should be of the form "Stop ..." (golint)

• go-flagz/watcher/watcher.go
  • Line 5: warning: package comment is detached; there should be no blank lines between it and the package statement (golint)
  • Line 82: warning: comment on exported method Watcher.Stop should be of the form "Stop ..." (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign90%

IneffAssign detects ineffectual assignments in Go code.

• go-flagz/dynfloat64_test.go
  • Line 63: warning: ineffectual assignment to x (ineffassign)
  • Line 73: warning: ineffectual assignment to x (ineffassign)

• go-flagz/dynint64_test.go
  • Line 61: warning: ineffectual assignment to x (ineffassign)
  • Line 71: warning: ineffectual assignment to x (ineffassign)

• go-flagz/dynstring_test.go
  • Line 63: warning: ineffectual assignment to x (ineffassign)
  • Line 73: warning: ineffectual assignment to x (ineffassign)

misspell96%

Misspell Finds commonly misspelled English words

• go-flagz/dynstring_test.go
  • Line 26: warning: "somethign" is a misspelling of "something" (misspell)