Report for github.com/adamwasila/slowjoe
A+ Excellent! Found 9 issues across 15 files
gofmt93%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
-
-
slowjoe/admin/admin.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!
gocyclo93%
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.
-
-
slowjoe/proxy.go
- Line 122: warning: cyclomatic complexity 19 of function (*Proxy).ListenAndLoop() is high (> 15) (gocyclo)
- Line 274: warning: cyclomatic complexity 16 of function (*connection).handleConnection() is high (> 15) (gocyclo)
golint53%
Golint is a linter for Go source code.
-
-
slowjoe/config/magic.go
- Line 4: warning: exported const DirUpstream should have comment (or a comment on this block) or be unexported (golint)
-
-
slowjoe/admin/admin.go
- Line 51: warning: exported function AddRoutes should have comment or be unexported (golint)
- Line 65: warning: exported function Assets should have comment or be unexported (golint)
- Line 73: warning: exported function Redirect should have comment or be unexported (golint)
- Line 79: warning: exported function ForTemplate should have comment or be unexported (golint)
- Line 123: warning: exported type Ui should have comment or be unexported (golint)
- Line 127: warning: exported type MenuItem should have comment or be unexported (golint)
- Line 133: warning: exported type ConnData should have comment or be unexported (golint)
- Line 143: warning: exported type AdminData should have comment or be unexported (golint)
- Line 152: warning: exported method ConnData.Since should have comment or be unexported (golint)
- Line 159: warning: exported method ConnData.Until should have comment or be unexported (golint)
- Line 166: warning: exported function NewAdminData should have comment or be unexported (golint)
- Line 172: warning: exported method AdminData.ConnectionOpened should have comment or be unexported (golint)
- Line 186: warning: exported method AdminData.ConnectionProgressed should have comment or be unexported (golint)
- Line 199: warning: exported method AdminData.ConnectionDelayed should have comment or be unexported (golint)
- Line 202: warning: exported method AdminData.ConnectionCompleted should have comment or be unexported (golint)
- Line 205: warning: exported method AdminData.ConnectionScheduledClose should have comment or be unexported (golint)
- Line 208: warning: exported method AdminData.ConnectionClosedUpstream should have comment or be unexported (golint)
- Line 211: warning: exported method AdminData.ConnectionClosedDownstream should have comment or be unexported (golint)
- Line 214: warning: exported method AdminData.ConnectionClosed should have comment or be unexported (golint)
- Line 229: warning: exported method AdminData.RLock should have comment or be unexported (golint)
- Line 233: warning: exported method AdminData.RUnlock should have comment or be unexported (golint)
-
-
slowjoe/instrumentation.go
- Line 18: warning: exported type Instrumentations should have comment or be unexported (golint)
- Line 20: warning: exported method Instrumentations.ConnectionOpened should have comment or be unexported (golint)
- Line 26: warning: exported method Instrumentations.ConnectionProgressed should have comment or be unexported (golint)
- Line 32: warning: exported method Instrumentations.ConnectionCompleted should have comment or be unexported (golint)
- Line 38: warning: exported method Instrumentations.ConnectionScheduledClose should have comment or be unexported (golint)
- Line 44: warning: exported method Instrumentations.ConnectionDelayed should have comment or be unexported (golint)
- Line 50: warning: exported method Instrumentations.ConnectionClosedUpstream should have comment or be unexported (golint)
- Line 56: warning: exported method Instrumentations.ConnectionClosedDownstream should have comment or be unexported (golint)
- Line 62: warning: exported method Instrumentations.ConnectionClosed should have comment or be unexported (golint)
-
-
slowjoe/logs.go
- Line 10: warning: exported type Logs should have comment or be unexported (golint)
- Line 14: warning: exported function DefaultLogs should have comment or be unexported (golint)
- Line 20: warning: exported method Logs.ConnectionOpened should have comment or be unexported (golint)
- Line 24: warning: exported method Logs.ConnectionProgressed should have comment or be unexported (golint)
- Line 28: warning: exported method Logs.ConnectionDelayed should have comment or be unexported (golint)
- Line 32: warning: exported method Logs.ConnectionCompleted should have comment or be unexported (golint)
- Line 42: warning: exported method Logs.ConnectionScheduledClose should have comment or be unexported (golint)
- Line 46: warning: exported method Logs.ConnectionClosedUpstream should have comment or be unexported (golint)
- Line 49: warning: exported method Logs.ConnectionClosedDownstream should have comment or be unexported (golint)
- Line 52: warning: exported method Logs.ConnectionClosed should have comment or be unexported (golint)
-
-
slowjoe/metrics.go
- Line 18: warning: exported type Metrics should have comment or be unexported (golint)
- Line 27: warning: exported method Metrics.ConnectionOpened should have comment or be unexported (golint)
- Line 31: warning: exported method Metrics.ConnectionProgressed should have comment or be unexported (golint)
- Line 35: warning: exported method Metrics.ConnectionDelayed should have comment or be unexported (golint)
- Line 38: warning: exported method Metrics.ConnectionCompleted should have comment or be unexported (golint)
- Line 41: warning: exported method Metrics.ConnectionScheduledClose should have comment or be unexported (golint)
- Line 44: warning: exported method Metrics.ConnectionClosedUpstream should have comment or be unexported (golint)
- Line 48: warning: exported method Metrics.ConnectionClosedDownstream should have comment or be unexported (golint)
- Line 52: warning: exported method Metrics.ConnectionClosed should have comment or be unexported (golint)
- Line 57: warning: exported method Metrics.Init should have comment or be unexported (golint)
-
-
slowjoe/proxy.go
- Line 13: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
- Line 21: warning: exported type Proxy should have comment or be unexported (golint)
- Line 49: warning: exported function Version should have comment or be unexported (golint)
- Line 56: warning: exported function Bind should have comment or be unexported (golint)
- Line 64: warning: exported function Upstream should have comment or be unexported (golint)
- Line 72: warning: exported function Config should have comment or be unexported (golint)
- Line 104: warning: exported function Instrument should have comment or be unexported (golint)
- Line 111: warning: exported function New should have comment or be unexported (golint)
- Line 122: warning: exported method Proxy.ListenAndLoop should have comment or be unexported (golint)
-
-
slowjoe/config/config.go
- Line 11: warning: exported type Config should have comment or be unexported (golint)
ineffassign100%
IneffAssign detects ineffectual assignments in Go code.
No problems detected. Good job!
misspell86%
Misspell Finds commonly misspelled English words
-
-
slowjoe/executor_test.go
- Line 169: warning: "succesfuly" is a misspelling of "successfully" (misspell)
- Line 195: warning: "inbetween" is a misspelling of "between" (misspell)
- Line 238: warning: "inbetween" is a misspelling of "between" (misspell)
-
-
slowjoe/gracefullquit_test.go
- Line 19: warning: "gracefull" is a misspelling of "graceful" (misspell)