Preparing report...

Report for github.com/roadrunner-server/sdk/v2

(v2.0.0-rc.1)

A+    Excellent!    Found 22 issues across 48 files

Tweet

gofmt100%

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

No problems detected. Good job!


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!


gocyclo97%

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.

    • bst/bst_test.go
    • Line 163: warning: cyclomatic complexity 18 of function BenchmarkBigSearchWithRemoves() is high (> 15) (gocyclo)

golint56%

Golint is a linter for Go source code.

    • worker/state.go
    • Line 40: warning: exported type StateImpl should have comment or be unexported (golint)
    • Line 109: warning: exported method StateImpl.LastUsed should have comment or be unexported (golint)
    • worker/sync_worker.go
    • Line 19: warning: exported type SyncWorkerImpl should have comment or be unexported (golint)
    • Line 229: warning: exported method SyncWorkerImpl.Pid should have comment or be unexported (golint)
    • Line 233: warning: exported method SyncWorkerImpl.Created should have comment or be unexported (golint)
    • Line 237: warning: exported method SyncWorkerImpl.State should have comment or be unexported (golint)
    • Line 241: warning: exported method SyncWorkerImpl.Start should have comment or be unexported (golint)
    • Line 245: warning: exported method SyncWorkerImpl.Wait should have comment or be unexported (golint)
    • Line 249: warning: exported method SyncWorkerImpl.Stop should have comment or be unexported (golint)
    • Line 253: warning: exported method SyncWorkerImpl.Kill should have comment or be unexported (golint)
    • Line 257: warning: exported method SyncWorkerImpl.Relay should have comment or be unexported (golint)
    • Line 261: warning: exported method SyncWorkerImpl.AttachRelay should have comment or be unexported (golint)
    • pool/supervisor_pool.go
    • Line 17: warning: exported const MB should have comment (or a comment on this block) or be unexported (golint)
    • Line 23: warning: exported type Supervised should have comment or be unexported (golint)
    • utils/network.go
    • Line 17: warning: exported const IPV4 should have comment (or a comment on this block) or be unexported (golint)
    • Line 21: warning: comment on exported function CreateListener should be of the form "CreateListener ..." (golint)
    • events/types.go
    • Line 7: warning: exported type EventBus should have comment or be unexported (golint)
    • Line 16: warning: exported type Event should have comment or be unexported (golint)
    • internal/protocol.go
    • Line 13: warning: exported type StopCommand should have comment or be unexported (golint)
    • Line 34: warning: exported function SendControl should have comment or be unexported (golint)
    • Line 76: warning: exported function Pid should have comment or be unexported (golint)
    • pool/static_pool.go
    • Line 26: warning: exported type Options should have comment or be unexported (golint)
    • Line 28: warning: exported type Command should have comment or be unexported (golint)
    • Line 113: warning: exported function WithLogger should have comment or be unexported (golint)
    • Line 129: warning: exported method StaticPool.RemoveWorker should have comment or be unexported (golint)
    • Line 172: warning: exported method StaticPool.Reset should have comment or be unexported (golint)
    • worker_watcher/container/channel/vec.go
    • Line 12: warning: exported type Vec should have comment or be unexported (golint)
    • Line 20: warning: exported function NewVector should have comment or be unexported (golint)
    • Line 98: warning: exported method Vec.Remove should have comment or be unexported (golint)
    • Line 100: warning: exported method Vec.Pop should have comment or be unexported (golint)
    • Line 125: warning: exported method Vec.Destroy should have comment or be unexported (golint)
    • priority_queue/binary_heap.go
    • Line 12: warning: exported type BinHeap should have comment or be unexported (golint)
    • Line 20: warning: exported function NewBinHeap should have comment or be unexported (golint)
    • Line 72: warning: exported method BinHeap.Len should have comment or be unexported (golint)
    • Line 76: warning: exported method BinHeap.Insert should have comment or be unexported (golint)
    • Line 106: warning: exported method BinHeap.ExtractMin should have comment or be unexported (golint)
    • worker/interface.go
    • Line 31: warning: exported type BaseProcess should have comment or be unexported (golint)
    • Line 67: warning: exported type SyncWorker should have comment or be unexported (golint)
    • pool/config.go
    • Line 53: warning: exported type SupervisorConfig should have comment or be unexported (golint)
    • worker_watcher/container/queue/queue.go
    • Line 17: warning: exported type Node should have comment or be unexported (golint)
    • Line 23: warning: exported type Queue should have comment or be unexported (golint)
    • Line 35: warning: exported function NewQueue should have comment or be unexported (golint)
    • Line 48: warning: exported method Queue.Push should have comment or be unexported (golint)
    • Line 70: warning: exported method Queue.Pop should have comment or be unexported (golint)
    • Line 94: warning: exported method Queue.Replace should have comment or be unexported (golint)
    • Line 98: warning: exported method Queue.Destroy should have comment or be unexported (golint)
    • bst/bst.go
    • Line 15: warning: exported function NewBST should have comment or be unexported (golint)
    • Line 55: warning: exported method BST.Contains should have comment or be unexported (golint)
    • Line 71: warning: exported method BST.Get should have comment or be unexported (golint)
    • Line 87: warning: exported method BST.Remove should have comment or be unexported (golint)
    • worker/worker.go
    • Line 18: warning: exported type Options should have comment or be unexported (golint)
    • Line 79: warning: exported function WithLog should have comment or be unexported (golint)
    • Line 127: warning: exported method Process.Start should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!