gofmt96%

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

• bricks/http/jsonapi/request_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• bricks/backend/redis/health_redis_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• bricks/maintenance/log/hlog/hlog.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• bricks/http/jsonapi/runtime/error.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• bricks/http/jsonapi/runtime/error_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• bricks/http/jsonapi/runtime/marshalling_test.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• bricks/tools/testserver/main.go
  • Line 1: warning: file is not gofmted with -s (gofmt)

• bricks/pkg/routine/redis_lock.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!

gocyclo96%

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.

• bricks/http/jsonapi/generator/generate_helper.go
  • Line 41: warning: cyclomatic complexity 29 of function (*typeGenerator).invoke() is high (> 15) (gocyclo)

• bricks/http/jsonapi/generator/generate_types.go
  • Line 58: warning: cyclomatic complexity 27 of function (*Generator).buildType() is high (> 15) (gocyclo)

• bricks/http/jsonapi/generator/generate_handler.go
  • Line 532: warning: cyclomatic complexity 24 of function (*Generator).buildHandler() is high (> 15) (gocyclo)

• bricks/http/jsonapi/response.go
  • Line 200: warning: cyclomatic complexity 71 of function visitModelNode() is high (> 15) (gocyclo)

• bricks/http/jsonapi/request.go
  • Line 147: warning: cyclomatic complexity 32 of function unmarshalNode() is high (> 15) (gocyclo)

• bricks/http/jsonapi/request_test.go
  • Line 1176: warning: cyclomatic complexity 17 of function TestUnmarshalNestedStruct() is high (> 15) (gocyclo)

• bricks/http/transport/dump_round_tripper.go
  • Line 96: warning: cyclomatic complexity 16 of function (*DumpRoundTripper).RoundTrip() is high (> 15) (gocyclo)

• bricks/maintenance/errors/raven/stacktrace.go
  • Line 125: warning: cyclomatic complexity 16 of function NewStacktraceFrame() is high (> 15) (gocyclo)

golint79%

Golint is a linter for Go source code.

• bricks/http/oauth2/oauth2.go
  • Line 20: warning: comment on exported type Middleware should be of the form "Middleware ..." (with optional leading article) (golint)
  • Line 26: warning: comment on exported function NewMiddleware should be of the form "NewMiddleware ..." (golint)
  • Line 177: warning: context.Context should be the first parameter of a function (golint)
  • Line 182: warning: comment on exported function BearerToken should be of the form "BearerToken ..." (golint)
  • Line 192: warning: comment on exported function WithBearerToken should be of the form "WithBearerToken ..." (golint)

• bricks/pkg/context/transfer.go
  • Line 29: warning: exported function TransferTracingContext should have comment or be unexported (golint)

• bricks/backend/couchdb/db.go
  • Line 18: warning: exported function DefaultDatabase should have comment or be unexported (golint)
  • Line 22: warning: exported function Database should have comment or be unexported (golint)
  • Line 71: warning: exported function Client should have comment or be unexported (golint)
  • Line 97: warning: exported function ParseConfig should have comment or be unexported (golint)

• bricks/backend/couchdb/health_check.go
  • Line 109: warning: exported type Doc should have comment or be unexported (golint)

• bricks/http/middleware/external_dependency.go
  • Line 37: warning: exported function AddExternalDependency should have comment or be unexported (golint)
  • Line 92: warning: exported method ExternalDependencyContext.AddDependency should have comment or be unexported (golint)

• bricks/maintenance/health/health.go
  • Line 29: warning: comment on exported function SetCustomReadinessCheck should be of the form "SetCustomReadinessCheck ..." (golint)

• bricks/http/transport/dump_round_tripper.go
  • Line 29: warning: exported type DumpRoundTripperOption should have comment or be unexported (golint)
  • Line 32: warning: exported const DumpRoundTripperOptionRequest should have comment (or a comment on this block) or be unexported (golint)

• bricks/backend/queue/rmq.go
  • Line 76: warning: exported type HealthCheck should have comment or be unexported (golint)

• bricks/maintenance/log/hlog/hlog.go
  • Line 211: warning: exported function WithValue should have comment or be unexported (golint)

• bricks/http/middleware/metrics.go
  • Line 62: warning: exported function Metrics should have comment or be unexported (golint)

• bricks/pkg/cache/testsuite/cache.go
  • Line 18: warning: exported type CacheTestSuite should have comment or be unexported (golint)
  • Line 23: warning: exported method CacheTestSuite.TestPut should have comment or be unexported (golint)
  • Line 91: warning: exported method CacheTestSuite.TestGet should have comment or be unexported (golint)
  • Line 194: warning: exported method CacheTestSuite.TestForget should have comment or be unexported (golint)

• bricks/maintenance/metric/jsonapi/jsonapi.go
  • Line 25: warning: exported const TypeRequest should have comment (or a comment on this block) or be unexported (golint)

• bricks/backend/couchdb/auth.go
  • Line 8: warning: exported type AuthTransport should have comment or be unexported (golint)
  • Line 13: warning: exported method AuthTransport.RoundTrip should have comment or be unexported (golint)
  • Line 18: warning: exported method AuthTransport.Transport should have comment or be unexported (golint)
  • Line 18: warning: receiver name rt should be consistent with previous receiver name l for AuthTransport (golint)
  • Line 22: warning: exported method AuthTransport.SetTransport should have comment or be unexported (golint)

• bricks/backend/objstore/objstore.go
  • Line 32: warning: exported function RegisterHealthchecks should have comment or be unexported (golint)
  • Line 36: warning: comment on exported function Client should be of the form "Client ..." (golint)
  • Line 41: warning: comment on exported function DefaultClientFromEnv should be of the form "DefaultClientFromEnv ..." (golint)

• bricks/pkg/isotime/isotime.go
  • Line 20: warning: exported function ParseISO8601 should have comment or be unexported (golint)

• bricks/maintenance/errors/raven/http.go
  • Line 13: warning: exported function NewHttp should have comment or be unexported (golint)
  • Line 48: warning: comment on exported type Http should be of the form "Http ..." (with optional leading article) (golint)
  • Line 64: warning: exported method Http.Class should have comment or be unexported (golint)
  • Line 66: warning: comment on exported function RecoveryHandler should be of the form "RecoveryHandler ..." (golint)

• bricks/maintenance/log/handler.go
  • Line 102: warning: exported function RequestIDHandler should have comment or be unexported (golint)

• bricks/http/oauth2/middleware/scopes_middleware.go
  • Line 17: warning: comment on exported type ScopesMiddleware should be of the form "ScopesMiddleware ..." (with optional leading article) (golint)
  • Line 23: warning: comment on exported function NewScopesMiddleware should be of the form "NewScopesMiddleware ..." (golint)

• bricks/pkg/redact/redact.go
  • Line 10: warning: exported type PatternRedactor should have comment or be unexported (golint)
  • Line 22: warning: exported method PatternRedactor.Mask should have comment or be unexported (golint)
  • Line 32: warning: comment on exported method PatternRedactor.AddPatterns should be of the form "AddPatterns ..." (golint)
  • Line 51: warning: exported method PatternRedactor.SetScheme should have comment or be unexported (golint)
  • Line 55: warning: exported method PatternRedactor.Clone should have comment or be unexported (golint)

• bricks/maintenance/health/servicehealthcheck/healthchecker.go
  • Line 25: warning: exported type HealthCheckFunc should have comment or be unexported (golint)
  • Line 27: warning: exported method HealthCheckFunc.HealthCheck should have comment or be unexported (golint)

• bricks/http/transport/default_transport.go
  • Line 21: warning: comment on exported function NewDefaultTransportChainWithExternalName should be of the form "NewDefaultTransportChainWithExternalName ..." (golint)

• bricks/maintenance/errors/raven/interfaces.go
  • Line 3: warning: comment on exported type Message should be of the form "Message ..." (with optional leading article) (golint)
  • Line 12: warning: exported method Message.Class should have comment or be unexported (golint)
  • Line 14: warning: comment on exported type Template should be of the form "Template ..." (with optional leading article) (golint)
  • Line 27: warning: exported method Template.Class should have comment or be unexported (golint)
  • Line 29: warning: comment on exported type User should be of the form "User ..." (with optional leading article) (golint)
  • Line 38: warning: exported method User.Class should have comment or be unexported (golint)
  • Line 40: warning: comment on exported type Query should be of the form "Query ..." (with optional leading article) (golint)
  • Line 49: warning: exported method Query.Class should have comment or be unexported (golint)

• bricks/backend/redis/redis.go
  • Line 187: warning: receiver name l should be consistent with previous receiver name lt for logtracer (golint)
  • Line 212: warning: receiver name l should be consistent with previous receiver name lt for logtracer (golint)
  • Line 216: warning: receiver name l should be consistent with previous receiver name lt for logtracer (golint)

• bricks/locale/strategy.go
  • Line 14: warning: comment on exported function NewFallbackStrategy should be of the form "NewFallbackStrategy ..." (golint)

• bricks/maintenance/errors/error.go
  • Line 141: warning: exported function HandleErrorNoStack should have comment or be unexported (golint)

• bricks/pkg/redact/default.go
  • Line 9: warning: exported var Default should have comment or be unexported (golint)

• bricks/pkg/redact/pattern.go
  • Line 24: warning: exported var PatternIBAN should have comment or be unexported (golint)
  • Line 32: warning: comment on exported var PatternCCVisa should be of the form "PatternCCVisa ..." (golint)
  • Line 35: warning: comment on exported var PatternCCMasterCard should be of the form "PatternCCMasterCard ..." (golint)
  • Line 38: warning: comment on exported var PatternCCAmericanExpress should be of the form "PatternCCAmericanExpress ..." (golint)
  • Line 41: warning: comment on exported var PatternCCDinersClub should be of the form "PatternCCDinersClub ..." (golint)
  • Line 44: warning: comment on exported var PatternCCDiscover should be of the form "PatternCCDiscover ..." (golint)
  • Line 47: warning: comment on exported var PatternCCJCB should be of the form "PatternCCJCB ..." (golint)

• bricks/http/transport/circuit_breaker_tripper.go
  • Line 34: warning: exported function NewDefaultCircuitBreakerTripper should have comment or be unexported (golint)
  • Line 40: warning: exported function NewCircuitBreakerTripper should have comment or be unexported (golint)

• bricks/http/transport/errors.go
  • Line 6: warning: exported var ErrRetryFailed should have comment or be unexported (golint)

• bricks/maintenance/errors/raven/errors.go
  • Line 24: warning: comment on exported function WrapWithExtra should be of the form "WrapWithExtra ..." (golint)
  • Line 32: warning: exported type ErrWithExtra should have comment or be unexported (golint)

• bricks/http/jsonapi/middleware/error_middleware.go
  • Line 49: warning: comment on exported function Error should be of the form "Error ..." (golint)

• bricks/locale/context.go
  • Line 32: warning: context.Context should be the first parameter of a function (golint)

• bricks/locale/locale.go
  • Line 4: warning: package comment should be of the form "Package locale ..." (golint)
  • Line 25: warning: exported var ErrNoTimezone should have comment or be unexported (golint)
  • Line 51: warning: comment on exported method Locale.HasLanguage should be of the form "HasLanguage ..." (golint)

• bricks/maintenance/log/sink.go
  • Line 201: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 208: warning: exported type SinkOption should have comment or be unexported (golint)
  • Line 210: warning: exported function Silent should have comment or be unexported (golint)
  • Line 216: warning: exported function CustomSize should have comment or be unexported (golint)

• bricks/maintenance/errors/raven/client.go
  • Line 38: warning: exported var ErrPacketDropped should have comment or be unexported (golint)
  • Line 45: warning: exported type Severity should have comment or be unexported (golint)
  • Line 56: warning: exported type Timestamp should have comment or be unexported (golint)
  • Line 58: warning: exported method Timestamp.MarshalJSON should have comment or be unexported (golint)
  • Line 62: warning: exported method Timestamp.UnmarshalJSON should have comment or be unexported (golint)
  • Line 62: warning: receiver name timestamp should be consistent with previous receiver name t for Timestamp (golint)
  • Line 72: warning: exported method Timestamp.Format should have comment or be unexported (golint)
  • Line 72: warning: receiver name timestamp should be consistent with previous receiver name t for Timestamp (golint)
  • Line 84: warning: exported type Culpriter should have comment or be unexported (golint)
  • Line 88: warning: exported type Transport should have comment or be unexported (golint)
  • Line 92: warning: exported type Extra should have comment or be unexported (golint)
  • Line 99: warning: exported type Tag should have comment or be unexported (golint)
  • Line 104: warning: exported type Tags should have comment or be unexported (golint)
  • Line 106: warning: exported method Tag.MarshalJSON should have comment or be unexported (golint)
  • Line 110: warning: exported method Tag.UnmarshalJSON should have comment or be unexported (golint)
  • Line 110: warning: receiver name t should be consistent with previous receiver name tag for Tag (golint)
  • Line 119: warning: exported method Tags.UnmarshalJSON should have comment or be unexported (golint)
  • Line 147: warning: comment on exported type Packet should be of the form "Packet ..." (with optional leading article) (golint)
  • Line 175: warning: exported type Breadcrumb should have comment or be unexported (golint)
  • Line 260: warning: exported method Packet.AddTags should have comment or be unexported (golint)
  • Line 279: warning: exported method Packet.JSON should have comment or be unexported (golint)
  • Line 346: warning: comment on exported var MaxQueueBuffer should be of the form "MaxQueueBuffer ..." (golint)
  • Line 443: warning: comment on exported var DefaultClient should be of the form "DefaultClient ..." (golint)
  • Line 446: warning: exported method Client.SetIgnoreErrors should have comment or be unexported (golint)
  • Line 465: warning: exported function SetIgnoreErrors should have comment or be unexported (golint)
  • Line 471: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 510: warning: comment on exported function SetDSN should be of the form "SetDSN ..." (golint)
  • Line 514: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 521: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 528: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 535: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 560: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 575: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 665: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 686: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 709: warning: comment on exported method Client.CaptureError should be of the form "CaptureError ..." (golint)
  • Line 711: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 733: warning: comment on exported function CaptureError should be of the form "CaptureError ..." (golint)
  • Line 740: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 768: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 806: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 846: warning: exported method Client.Close should have comment or be unexported (golint)
  • Line 846: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 850: warning: exported function Close should have comment or be unexported (golint)
  • Line 853: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 860: warning: exported method Client.URL should have comment or be unexported (golint)
  • Line 860: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 867: warning: exported function URL should have comment or be unexported (golint)
  • Line 869: warning: exported method Client.ProjectID should have comment or be unexported (golint)
  • Line 869: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 876: warning: exported function ProjectID should have comment or be unexported (golint)
  • Line 878: warning: exported method Client.Release should have comment or be unexported (golint)
  • Line 878: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 885: warning: exported function Release should have comment or be unexported (golint)
  • Line 887: warning: exported function IncludePaths should have comment or be unexported (golint)
  • Line 889: warning: exported method Client.IncludePaths should have comment or be unexported (golint)
  • Line 889: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 896: warning: exported function SetIncludePaths should have comment or be unexported (golint)
  • Line 898: warning: exported method Client.SetIncludePaths should have comment or be unexported (golint)
  • Line 898: warning: receiver name client should be consistent with previous receiver name c for Client (golint)
  • Line 905: warning: exported method Client.SetUserContext should have comment or be unexported (golint)
  • Line 911: warning: exported method Client.SetHttpContext should have comment or be unexported (golint)
  • Line 917: warning: exported method Client.SetTagsContext should have comment or be unexported (golint)
  • Line 923: warning: exported method Client.ClearContext should have comment or be unexported (golint)
  • Line 929: warning: exported function SetUserContext should have comment or be unexported (golint)
  • Line 930: warning: exported function SetHttpContext should have comment or be unexported (golint)
  • Line 931: warning: exported function SetTagsContext should have comment or be unexported (golint)
  • Line 932: warning: exported function ClearContext should have comment or be unexported (golint)
  • Line 940: warning: exported method HTTPTransport.Send should have comment or be unexported (golint)

• bricks/maintenance/errors/raven/exception.go
  • Line 10: warning: exported function NewException should have comment or be unexported (golint)
  • Line 23: warning: comment on exported type Exception should be of the form "Exception ..." (with optional leading article) (golint)
  • Line 34: warning: exported method Exception.Class should have comment or be unexported (golint)
  • Line 36: warning: exported method Exception.Culprit should have comment or be unexported (golint)
  • Line 50: warning: exported method Exceptions.Class should have comment or be unexported (golint)

• bricks/backend/couchdb/couchdb.go
  • Line 8: warning: exported type Config should have comment or be unexported (golint)

• bricks/pkg/redact/scheme.go
  • Line 8: warning: exported type RedactionScheme should have comment or be unexported (golint)
  • Line 30: warning: comment on exported function RedactionSchemeKeepLastJWTNoSignature should be of the form "RedactionSchemeKeepLastJWTNoSignature ..." (golint)

• bricks/http/transport/locale_round_tripper.go
  • Line 32: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• bricks/maintenance/errors/raven/stacktrace.go
  • Line 20: warning: comment on exported type Stacktrace should be of the form "Stacktrace ..." (with optional leading article) (golint)
  • Line 26: warning: exported method Stacktrace.Class should have comment or be unexported (golint)
  • Line 28: warning: exported method Stacktrace.Culprit should have comment or be unexported (golint)
  • Line 38: warning: exported type StacktraceFrame should have comment or be unexported (golint)
  • Line 54: warning: comment on exported function GetOrNewStacktrace should be of the form "GetOrNewStacktrace ..." (golint)
  • Line 77: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 82: warning: comment on exported function NewStacktrace should be of the form "NewStacktrace ..." (golint)
  • Line 117: warning: comment on exported function NewStacktraceFrame should be of the form "NewStacktraceFrame ..." (golint)

• bricks/maintenance/errors/raven/writer.go
  • Line 3: warning: exported type Writer should have comment or be unexported (golint)

• bricks/backend/postgres/postgres.go
  • Line 27: warning: exported type Config should have comment or be unexported (golint)

• bricks/locale/http.go
  • Line 9: warning: exported const HeaderAcceptLanguage should have comment (or a comment on this block) or be unexported (golint)

• bricks/tools/testserver/main.go
  • Line 37: warning: exported type OauthBackend should have comment or be unexported (golint)
  • Line 39: warning: exported method OauthBackend.IntrospectToken should have comment or be unexported (golint)
  • Line 48: warning: exported type TestService should have comment or be unexported (golint)
  • Line 50: warning: exported method TestService.GetTest 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!

misspell94%

Misspell Finds commonly misspelled English words

• bricks/backend/postgres/metrics.go
  • Line 147: warning: "decribed" is a misspelling of "described" (misspell)

• bricks/backend/postgres/postgres.go
  • Line 35: warning: "availaible" is a misspelling of "available" (misspell)

• bricks/http/longpoll/longpoll.go
  • Line 34: warning: "constaints" is a misspelling of "constraints" (misspell)

• bricks/internal/service/service.go
  • Line 119: warning: "installes" is a misspelling of "installs" (misspell)

• bricks/http/oauth2/oauth2_test.go
  • Line 237: warning: "succesful" is a misspelling of "successful" (misspell)

• bricks/backend/objstore/health_objstore.go
  • Line 130: warning: "concider" is a misspelling of "consider" (misspell)

• bricks/backend/couchdb/health_check.go
  • Line 127: warning: "concider" is a misspelling of "consider" (misspell)

• bricks/http/transport/chainable.go
  • Line 45: warning: "overriden" is a misspelling of "overridden" (misspell)

• bricks/pkg/redact/default.go
  • Line 6: warning: "concidered" is a misspelling of "considered" (misspell)

• bricks/http/router.go
  • Line 61: warning: "accross" is a misspelling of "across" (misspell)
  • Line 64: warning: "accessable" is a misspelling of "accessible" (misspell)

• bricks/http/jsonapi/errors_test.go
  • Line 59: warning: "nto" is a misspelling of "not" (misspell)