Preparing report...

Report for github.com/unlaunch/go-sdk

A    Great!    Found 24 issues across 56 files

Tweet

gofmt80%

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


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!


golint73%

Golint is a linter for Go source code.

    • go-sdk/unlaunchio/service/api/eventcountaggregator.go
    • Line 13: warning: exported type EventsCountAggregator should have comment or be unexported (golint)
    • Line 18: warning: exported type SimpleEventsCountAggregator should have comment or be unexported (golint)
    • Line 29: warning: exported method SimpleEventsCountAggregator.Shutdown should have comment or be unexported (golint)
    • Line 88: warning: exported method SimpleEventsCountAggregator.Record should have comment or be unexported (golint)
    • Line 108: warning: exported function NewEventsCountAggregator should have comment or be unexported (golint)
    • go-sdk/unlaunchio/engine/murmur3hash.go
    • Line 18: warning: a blank import should be only in a main or test package, or have a comment justifying it (golint)
    • Line 31: warning: exported type Digest should have comment or be unexported (golint)
    • Line 38: warning: comment on exported const Size should be of the form "Size ..." (golint)
    • Line 66: warning: comment on exported method Digest.Size should be of the form "Size ..." (golint)
    • Line 69: warning: comment on exported method Digest.BlockSize should be of the form "BlockSize ..." (golint)
    • Line 128: warning: comment on exported method Digest.Sum should be of the form "Sum ..." (golint)
    • Line 136: warning: comment on exported method Digest.Sum32 should be of the form "Sum32 ..." (golint)
    • Line 143: warning: comment on exported function Murmur32Hash should be of the form "Murmur32Hash ..." (golint)
    • go-sdk/unlaunchio/util/typehelper.go
    • Line 85: warning: exported function CovertToMap should have comment or be unexported (golint)
    • Line 94: warning: exported function JavaMillsToUnixTime should have comment or be unexported (golint)
    • Line 98: warning: exported function UnixTimeToJavaMillis should have comment or be unexported (golint)
    • go-sdk/unlaunchio/engine/attributes/targeting.go
    • Line 3: warning: exported type TargetingRuleCondition should have comment or be unexported (golint)
    • Line 6: warning: exported method TargetingRuleCondition.Apply should have comment or be unexported (golint)
    • Line 26: warning: exported function NewTargetingRule should have comment or be unexported (golint)
    • go-sdk/unlaunchio/service/api/eventrecorder.go
    • Line 13: warning: exported type EventsRecorder should have comment or be unexported (golint)
    • Line 18: warning: exported type SimpleEventsRecorder should have comment or be unexported (golint)
    • Line 75: warning: exported method SimpleEventsRecorder.Shutdown should have comment or be unexported (golint)
    • Line 85: warning: exported method SimpleEventsRecorder.Record should have comment or be unexported (golint)
    • Line 108: warning: exported function NewHTTPEventsRecorder should have comment or be unexported (golint)
    • go-sdk/unlaunchio/engine/evaluator.go
    • Line 15: warning: exported type Evaluator should have comment or be unexported (golint)
    • Line 19: warning: exported type SimpleEvaluator should have comment or be unexported (golint)
    • Line 23: warning: exported function NewEvaluator should have comment or be unexported (golint)
    • Line 29: warning: exported method SimpleEvaluator.Evaluate should have comment or be unexported (golint)
    • Line 76: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • go-sdk/unlaunchio/util/globalconstants.go
    • Line 3: warning: exported const SDKKeyHelpMessage should have comment or be unexported (golint)
    • Line 7: warning: exported const MaxUint should have comment or be unexported (golint)
    • Line 8: warning: exported const MaxInt should have comment or be unexported (golint)
    • go-sdk/unlaunchio/dtos/features.go
    • Line 3: warning: exported type Feature should have comment or be unexported (golint)
    • Line 12: warning: exported method Feature.Enabled should have comment or be unexported (golint)
    • Line 19: warning: exported method Feature.DefaultRule should have comment or be unexported (golint)
    • Line 30: warning: exported method Feature.VariationByID should have comment or be unexported (golint)
    • Line 39: warning: exported type Variation should have comment or be unexported (golint)
    • Line 46: warning: exported type Rollout should have comment or be unexported (golint)
    • Line 52: warning: exported type ByVariationID should have comment or be unexported (golint)
    • Line 58: warning: exported type Rule should have comment or be unexported (golint)
    • Line 65: warning: exported type ByRulePriority should have comment or be unexported (golint)
    • Line 71: warning: exported type Condition should have comment or be unexported (golint)
    • Line 80: warning: exported type Data should have comment or be unexported (golint)
    • Line 86: warning: exported type TopLevelEnvelope should have comment or be unexported (golint)
    • Line 90: warning: exported type UnlaunchFeature should have comment or be unexported (golint)
    • go-sdk/unlaunchio/service/httpfeaturestore.go
    • Line 14: warning: exported type HTTPFeatureStore should have comment or be unexported (golint)
    • Line 22: warning: exported method HTTPFeatureStore.Shutdown should have comment or be unexported (golint)
    • Line 83: warning: exported method HTTPFeatureStore.GetFeature should have comment or be unexported (golint)
    • Line 90: warning: exported method HTTPFeatureStore.IsReady should have comment or be unexported (golint)
    • Line 93: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 98: warning: exported method HTTPFeatureStore.Ready should have comment or be unexported (golint)
    • Line 115: warning: exported function NewHTTPFeatureStore 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!