gofmt98%

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

• cn-infra/db/cryptodata/wrapper.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• cn-infra/examples/tutorials/06_plugin_lookup/plugin1.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• cn-infra/examples/tutorials/06_plugin_lookup/plugin2.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• cn-infra/exec/supervisor/logger.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• cn-infra/utils/redact/redact.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!

gocyclo98%

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.

• cn-infra/db/keyval/etcd/config.go
  • Line 88: warning: cyclomatic complexity 18 of function ConfigToClient() is high (> 15) (gocyclo)

• cn-infra/examples/process-manager-plugin/advanced-scenario/main.go
  • Line 88: warning: cyclomatic complexity 17 of function (*PMExample).advancedExample() is high (> 15) (gocyclo)

• cn-infra/examples/redis-lib/simple/simple.go
  • Line 125: warning: cyclomatic complexity 19 of function (*SimpleRedis).start() is high (> 15) (gocyclo)

• cn-infra/exec/processmanager/process_impl.go
  • Line 230: warning: cyclomatic complexity 19 of function (*Process).watch() is high (> 15) (gocyclo)

• cn-infra/exec/processmanager/status/status.go
  • Line 142: warning: cyclomatic complexity 58 of function (*Reader).parse() is high (> 15) (gocyclo)

golint97%

Golint is a linter for Go source code.

• cn-infra/utils/ratelimit/limiters.go
  • Line 30: warning: exported function NewLimiter should have comment or be unexported (golint)
  • Line 58: warning: exported method Limiters.Allow should have comment or be unexported (golint)

• cn-infra/utils/redact/string.go
  • Line 24: warning: exported method MaskedString.Redacted should have comment or be unexported (golint)
  • Line 32: warning: exported method MaskedString.MarshalText should have comment or be unexported (golint)

• cn-infra/logging/logrus/entry.go
  • Line 75: warning: exported method Entry.Log should have comment or be unexported (golint)
  • Line 81: warning: exported method Entry.Logf should have comment or be unexported (golint)
  • Line 87: warning: exported method Entry.Logln should have comment or be unexported (golint)

• cn-infra/logging/logrus/formatter.go
  • Line 54: warning: exported type Formatter should have comment or be unexported (golint)
  • Line 62: warning: exported function NewFormatter should have comment or be unexported (golint)
  • Line 72: warning: exported method Formatter.Format should have comment or be unexported (golint)

• cn-infra/logging/log_api.go
  • Line 35: warning: exported var Trace should have comment or be unexported (golint)

• cn-infra/utils/redact/proto.go
  • Line 38: warning: exported function ContainsRedacted should have comment or be unexported (golint)

• cn-infra/logging/logrus/logger.go
  • Line 88: warning: exported method Logger.GetName should have comment or be unexported (golint)
  • Line 136: warning: exported method Logger.SetLevel should have comment or be unexported (golint)
  • Line 140: warning: exported method Logger.GetLevel should have comment or be unexported (golint)
  • Line 144: warning: exported method Logger.AddHook should have comment or be unexported (golint)
  • Line 148: warning: exported method Logger.SetOutput should have comment or be unexported (golint)
  • Line 152: warning: exported method Logger.SetFormatter should have comment or be unexported (golint)
  • Line 156: warning: exported method Logger.SetReportCaller should have comment or be unexported (golint)
  • Line 160: warning: exported method Logger.WithError should have comment or be unexported (golint)
  • Line 167: warning: exported method Logger.WithContext should have comment or be unexported (golint)
  • Line 217: warning: exported method Logger.Log should have comment or be unexported (golint)
  • Line 223: warning: exported method Logger.Logf should have comment or be unexported (golint)
  • Line 229: warning: exported method Logger.Logln should have comment or be unexported (golint)

• cn-infra/rpc/rest/security/rest_security.go
  • Line 37: warning: exported var ErrPermissionDenied should have comment or be unexported (golint)

• cn-infra/rpc/rest/middleware.go
  • Line 28: warning: exported const HeaderKeyAuthUsername should have comment (or a comment on this block) or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign98%

IneffAssign detects ineffectual assignments in Go code.

• cn-infra/db/keyval/filedb/filesystem/filesystem.go
  • Line 114: warning: ineffectual assignment to err (ineffassign)
  • Line 176: warning: ineffectual assignment to files (ineffassign)

• cn-infra/db/keyval/redis/bytes_suite_test.go
  • Line 341: warning: ineffectual assignment to err (ineffassign)
  • Line 380: warning: ineffectual assignment to err (ineffassign)

• cn-infra/db/sql/cassandra/cassa_list_values_test.go
  • Line 38: warning: ineffectual assignment to err (ineffassign)
  • Line 62: warning: ineffectual assignment to err (ineffassign)
  • Line 83: warning: ineffectual assignment to err (ineffassign)
  • Line 224: warning: ineffectual assignment to err (ineffassign)
  • Line 246: warning: ineffectual assignment to err (ineffassign)

• cn-infra/db/keyval/etcd/etcd_integration_test.go
  • Line 374: warning: ineffectual assignment to data (ineffassign)
  • Line 394: warning: ineffectual assignment to data (ineffassign)
  • Line 467: warning: ineffectual assignment to modRev (ineffassign)

• cn-infra/db/keyval/etcd/etcd_suite_test.go
  • Line 184: warning: ineffectual assignment to result (ineffassign)
  • Line 184: warning: ineffectual assignment to found (ineffassign)
  • Line 227: warning: ineffectual assignment to response (ineffassign)
  • Line 243: warning: ineffectual assignment to result (ineffassign)

• cn-infra/utils/addrs/addr_test.go
  • Line 66: warning: ineffectual assignment to ipv6 (ineffassign)
  • Line 69: warning: ineffectual assignment to ipv6 (ineffassign)
  • Line 72: warning: ineffectual assignment to ipv6 (ineffassign)

misspell97%

Misspell Finds commonly misspelled English words

• cn-infra/rpc/grpc/plugin_impl_grpc.go
  • Line 55: warning: "auther" is a misspelling of "author" (misspell)

• cn-infra/rpc/rest/config.go
  • Line 101: warning: "globaly" is a misspelling of "globally" (misspell)

• cn-infra/db/keyval/consul/plugin.go
  • Line 107: warning: "beacuse" is a misspelling of "because" (misspell)

• cn-infra/datasync/kvdbsync/watch_impl.go
  • Line 110: warning: "ommited" is a misspelling of "omitted" (misspell)

• cn-infra/examples/tutorials/01_hello-world/main.go
  • Line 52: warning: "initalized" is a misspelling of "initialized" (misspell)

• cn-infra/exec/processmanager/status/status.go
  • Line 58: warning: "Proces" is a misspelling of "Process" (misspell)

• cn-infra/logging/logmanager/hooks.go
  • Line 32: warning: "registy" is a misspelling of "registry" (misspell)

• cn-infra/db/keyval/etcd/config.go
  • Line 113: warning: "compatiblity" is a misspelling of "compatibility" (misspell)

• cn-infra/db/keyval/etcd/plugin_impl_etcd.go
  • Line 279: warning: "beacuse" is a misspelling of "because" (misspell)