Report for github.com/tipsio/tips
A+ Excellent! Found 9 issues across 16 files
gofmt87%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
-
-
tips/tipsd/server.go
- Line 1: warning: file is not gofmted with -s (gofmt)
-
-
tips/store/pubsub/pubsub_test.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!
golint68%
Golint is a linter for Go source code.
-
-
tips/conf/conf.go
- Line 3: warning: exported type Tips should have comment or be unexported (golint)
- Line 11: warning: exported type Server should have comment or be unexported (golint)
- Line 18: warning: exported type Tikv should have comment or be unexported (golint)
- Line 22: warning: exported type Logger should have comment or be unexported (golint)
- Line 30: warning: exported type TikvLogger should have comment or be unexported (golint)
- Line 37: warning: comment on exported type Status should be of the form "Status ..." (with optional leading article) (golint)
-
-
tips/store/pubsub/pubsub.go
- Line 25: warning: exported var ErrNotFound should have comment or be unexported (golint)
- Line 229: warning: comment on exported method Transaction.CreateSubscription should be of the form "CreateSubscription ..." (golint)
-
-
tips/metrics/metrics.go
- Line 30: warning: exported type Metrics should have comment or be unexported (golint)
-
-
tips/tipsd/server.go
- Line 94: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
-
-
tips/tipsd/tipsd.go
- Line 32: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 51: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 67: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 103: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 124: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 144: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 167: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 217: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 238: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
- Line 259: warning: receiver name t should be consistent with previous receiver name s for Server (golint)
ineffassign87%
IneffAssign detects ineffectual assignments in Go code.
-
-
tips/store/pubsub/pubsub_test.go
- Line 549: warning: ineffectual assignment to err (ineffassign)
-
-
tips/tipsd/server_test.go
- Line 125: warning: ineffectual assignment to body (ineffassign)
- Line 157: warning: ineffectual assignment to body (ineffassign)
- Line 160: warning: ineffectual assignment to body (ineffassign)
misspell81%
Misspell Finds commonly misspelled English words
-
-
tips/tips.go
- Line 116: warning: "destorys" is a misspelling of "destroys" (misspell)
-
-
tips/metrics/metrics.go
- Line 61: warning: "subscribtions" is a misspelling of "subscriptions" (misspell)
- Line 63: warning: "subscribtion" is a misspelling of "subscription" (misspell)
-
-
tips/metrics/server.go
- Line 46: warning: "sucess" is a misspelling of "success" (misspell)
- Line 61: warning: "sucess" is a misspelling of "success" (misspell)