gofmt93%

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

• util/network_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• util/network_windows.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• util/network_windows_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• service/http/errors.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• service/http/errors_windows.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• osutil/isolate.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• osutil/isolate_win.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• util/network.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!

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.

• service/http/config.go
  • Line 218: warning: cyclomatic complexity 19 of function (*Config).Valid() is high (> 15) (gocyclo)

• cmd/util/config.go
  • Line 35: warning: cyclomatic complexity 23 of function LoadConfig() is high (> 15) (gocyclo)

• service/http/service.go
  • Line 102: warning: cyclomatic complexity 22 of function (*Service).Serve() is high (> 15) (gocyclo)

golint91%

Golint is a linter for Go source code.

• service/reload/service.go
  • Line 17: warning: exported type Service should have comment or be unexported (golint)
  • Line 81: warning: exported method Service.Serve should have comment or be unexported (golint)
  • Line 160: warning: exported method Service.Stop should have comment or be unexported (golint)

• service/reload/watcher.go
  • Line 12: warning: exported var ErrorSkip should have comment or be unexported (golint)
  • Line 13: warning: error var NoWalkerConfig should have name of the form ErrFoo (golint)
  • Line 13: warning: exported var NoWalkerConfig should have comment or be unexported (golint)
  • Line 28: warning: exported type WatcherConfig should have comment or be unexported (golint)
  • Line 51: warning: exported type Watcher should have comment or be unexported (golint)
  • Line 198: warning: exported method Watcher.StartPolling should have comment or be unexported (golint)
  • Line 370: warning: exported method Watcher.Stop should have comment or be unexported (golint)

• service/metrics/config.go
  • Line 19: warning: exported type NamedCollector should have comment or be unexported (golint)

• static_pool.go
  • Line 130: warning: exported var ErrAllocateWorker should have comment or be unexported (golint)
  • Line 189: warning: exported var ErrPoolStopped should have comment or be unexported (golint)
  • Line 190: warning: exported var ErrWorkerAllocateTimeout should have comment or be unexported (golint)
  • Line 191: warning: exported var ErrAllWorkersAreDead should have comment or be unexported (golint)

• worker.go
  • Line 156: warning: exported var ErrEmptyPayload should have comment or be unexported (golint)
  • Line 157: warning: exported var ErrWorkerNotReady should have comment or be unexported (golint)

• service/http/service.go
  • Line 35: warning: error var couldNotAppendPemError should have name of the form errFoo (golint)

• service/reload/config.go
  • Line 23: warning: exported type ServiceConfig should have comment or be unexported (golint)

• service/gzip/service.go
  • Line 14: warning: error var httpNotInitialized should have name of the form errFoo (golint)
  • Line 16: warning: exported type Service should have comment or be unexported (golint)
  • Line 20: warning: exported method Service.Init should have comment or be unexported (golint)

• server.go
  • Line 136: warning: exported var ErrNoAssociatedPool should have comment or be unexported (golint)

• service/limit/service_test.go
  • Line 495: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

misspell99%

Misspell Finds commonly misspelled English words

• service/http/handler.go
  • Line 208: warning: "addres" is a misspelling of "address" (misspell)