Report for github.com/joinself/self-go-sdk
A+ Excellent! Found 14 issues across 60 files
gofmt96%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
-
-
self-go-sdk/pkg/ntp/time.go
- Line 1: warning: file is not gofmted with -s (gofmt)
-
-
self-go-sdk/fact/fact.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!
gocyclo90%
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.
-
-
self-go-sdk/fact/request.go
- Line 422: warning: cyclomatic complexity 21 of function (*Service).FactResponse() is high (> 15) (gocyclo)
-
-
self-go-sdk/authentication/authenticate.go
- Line 236: warning: cyclomatic complexity 19 of function (*Service).authenticationResponse() is high (> 15) (gocyclo)
-
-
self-go-sdk/config.go
- Line 100: warning: cyclomatic complexity 18 of function (*Config).load() is high (> 15) (gocyclo)
-
-
self-go-sdk/pkg/siggraph/graph.go
- Line 42: warning: cyclomatic complexity 28 of function (*SignatureGraph).Execute() is high (> 15) (gocyclo)
-
-
self-go-sdk/pkg/transport/support_test.go
- Line 158: warning: cyclomatic complexity 20 of function (*testmsgserver).testHandler() is high (> 15) (gocyclo)
-
-
self-go-sdk/pkg/transport/websocket.go
- Line 396: warning: cyclomatic complexity 19 of function (*Websocket).reader() is high (> 15) (gocyclo)
golint90%
Golint is a linter for Go source code.
-
-
self-go-sdk/pkg/ntp/time.go
- Line 13: warning: exported var TimeFunc should have comment or be unexported (golint)
- Line 23: warning: exported function NewTime should have comment or be unexported (golint)
- Line 29: warning: exported method Time.Now should have comment or be unexported (golint)
-
-
self-go-sdk/pkg/pqueue/pqueue.go
- Line 37: warning: comment on exported method Queue.PopWithPrioriry should be of the form "PopWithPrioriry ..." (golint)
-
-
self-go-sdk/authentication/authenticate.go
- Line 21: warning: exported var RequestAuthentication should have comment or be unexported (golint)
-
-
self-go-sdk/messaging/messaging.go
- Line 21: warning: exported var ErrBadJSONPayload should have comment or be unexported (golint)
-
-
self-go-sdk/fact/fact.go
- Line 15: warning: exported var SourcePassport should have comment or be unexported (golint)
-
-
self-go-sdk/fact/request.go
- Line 22: warning: exported var ErrBadJSONPayload should have comment or be unexported (golint)
- Line 98: warning: exported type QRConfig should have comment or be unexported (golint)
ineffassign96%
IneffAssign detects ineffectual assignments in Go code.
-
-
self-go-sdk/messaging/acl_test.go
- Line 105: warning: ineffectual assignment to err (ineffassign)
-
-
self-go-sdk/fact/request_test.go
- Line 88: warning: ineffectual assignment to err (ineffassign)
- Line 111: warning: ineffectual assignment to err (ineffassign)
- Line 285: warning: ineffectual assignment to err (ineffassign)
- Line 308: warning: ineffectual assignment to err (ineffassign)
- Line 411: warning: ineffectual assignment to err (ineffassign)
- Line 434: warning: ineffectual assignment to err (ineffassign)
- Line 539: warning: ineffectual assignment to err (ineffassign)
- Line 562: warning: ineffectual assignment to err (ineffassign)
- Line 667: warning: ineffectual assignment to err (ineffassign)
- Line 690: warning: ineffectual assignment to err (ineffassign)
- Line 793: warning: ineffectual assignment to err (ineffassign)
- Line 816: warning: ineffectual assignment to err (ineffassign)
- Line 919: warning: ineffectual assignment to err (ineffassign)
- Line 942: warning: ineffectual assignment to err (ineffassign)
- Line 1045: warning: ineffectual assignment to err (ineffassign)
- Line 1068: warning: ineffectual assignment to err (ineffassign)
- Line 1171: warning: ineffectual assignment to err (ineffassign)
- Line 1194: warning: ineffectual assignment to err (ineffassign)
- Line 1297: warning: ineffectual assignment to err (ineffassign)
- Line 1320: warning: ineffectual assignment to err (ineffassign)
misspell91%
Misspell Finds commonly misspelled English words
-
-
self-go-sdk/fact/request_test.go
- Line 1439: warning: "assertation" is a misspelling of "assertion" (misspell)
- Line 1676: warning: "assertation" is a misspelling of "assertion" (misspell)
- Line 1781: warning: "assertation" is a misspelling of "assertion" (misspell)
- Line 1886: warning: "assertation" is a misspelling of "assertion" (misspell)
- Line 1991: warning: "assertation" is a misspelling of "assertion" (misspell)
- Line 2097: warning: "assertation" is a misspelling of "assertion" (misspell)
- Line 2202: warning: "assertation" is a misspelling of "assertion" (misspell)
- Line 2308: warning: "assertation" is a misspelling of "assertion" (misspell)
-
-
self-go-sdk/pkg/crypto/crypto.go
- Line 43: warning: "retreive" is a misspelling of "retrieve" (misspell)
-
-
self-go-sdk/messaging/acl.go
- Line 54: warning: "exisitng" is a misspelling of "existing" (misspell)
-
-
self-go-sdk/authentication/authenticate.go
- Line 39: warning: "specifed" is a misspelling of "specified" (misspell)
-
-
self-go-sdk/fact/request.go
- Line 44: warning: "specifed" is a misspelling of "specified" (misspell)
- Line 104: warning: "paramters" is a misspelling of "parameters" (misspell)