Preparing report...

Report for github.com/lyft/flytestdlib

A+    Excellent!    Found 59 issues across 123 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!


gocyclo99%

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.


golint52%

Golint is a linter for Go source code.

    • flytestdlib/cache/auto_refresh.go
    • Line 22: warning: exported type ItemID should have comment or be unexported (golint)
    • Line 23: warning: exported type Batch should have comment or be unexported (golint)
    • Line 26: warning: exported const ErrNotFound should have comment (or a comment on this block) or be unexported (golint)
    • Line 55: warning: exported type Item should have comment or be unexported (golint)
    • Line 57: warning: comment on exported type ItemWrapper should be of the form "ItemWrapper ..." (with optional leading article) (golint)
    • Line 63: warning: comment on exported type ItemSyncResponse should be of the form "ItemSyncResponse ..." (with optional leading article) (golint)
    • Line 70: warning: comment on exported type SyncAction should be of the form "SyncAction ..." (with optional leading article) (golint)
    • Line 74: warning: exported const Unchanged should have comment (or a comment on this block) or be unexported (golint)
    • Line 76: warning: comment on exported const Update should be of the form "Update ..." (golint)
    • Line 80: warning: comment on exported type SyncFunc should be of the form "SyncFunc ..." (with optional leading article) (golint)
    • Line 87: warning: comment on exported type CreateBatchesFunc should be of the form "CreateBatchesFunc ..." (with optional leading article) (golint)
    • Line 127: warning: exported function SingleItemBatches should have comment or be unexported (golint)
    • Line 290: warning: comment on exported function NewAutoRefreshBatchedCache should be of the form "NewAutoRefreshBatchedCache ..." (golint)
    • Line 314: warning: comment on exported function NewAutoRefreshCache should be of the form "NewAutoRefreshCache ..." (golint)
    • flytestdlib/logger/logger.go
    • Line 1: warning: package comment should be of the form "Package logger ..." (golint)
    • Line 83: warning: comment on exported function GetLogWriter should be of the form "GetLogWriter ..." (golint)
    • Line 89: warning: exported function WithIndent should have comment or be unexported (golint)
    • Line 102: warning: comment on exported function IsLoggable should be of the form "IsLoggable ..." (golint)
    • Line 233: warning: exported type NoopLogger should have comment or be unexported (golint)
    • Line 236: warning: exported method NoopLogger.WithField should have comment or be unexported (golint)
    • Line 240: warning: exported method NoopLogger.WithFields should have comment or be unexported (golint)
    • Line 244: warning: exported method NoopLogger.WithError should have comment or be unexported (golint)
    • Line 248: warning: exported method NoopLogger.Debugf should have comment or be unexported (golint)
    • Line 251: warning: exported method NoopLogger.Infof should have comment or be unexported (golint)
    • Line 254: warning: exported method NoopLogger.Warnf should have comment or be unexported (golint)
    • Line 257: warning: exported method NoopLogger.Warningf should have comment or be unexported (golint)
    • Line 260: warning: exported method NoopLogger.Errorf should have comment or be unexported (golint)
    • Line 263: warning: exported method NoopLogger.Debug should have comment or be unexported (golint)
    • Line 266: warning: exported method NoopLogger.Info should have comment or be unexported (golint)
    • Line 269: warning: exported method NoopLogger.Warn should have comment or be unexported (golint)
    • Line 272: warning: exported method NoopLogger.Warning should have comment or be unexported (golint)
    • Line 278: warning: exported method NoopLogger.Debugln should have comment or be unexported (golint)
    • Line 281: warning: exported method NoopLogger.Infoln should have comment or be unexported (golint)
    • Line 284: warning: exported method NoopLogger.Warnln should have comment or be unexported (golint)
    • Line 287: warning: exported method NoopLogger.Warningln should have comment or be unexported (golint)
    • Line 290: warning: exported method NoopLogger.Errorln should have comment or be unexported (golint)
    • Line 293: warning: exported method NoopLogger.Print should have comment or be unexported (golint)
    • Line 296: warning: exported method NoopLogger.Printf should have comment or be unexported (golint)
    • Line 299: warning: exported method NoopLogger.Println should have comment or be unexported (golint)
    • Line 302: warning: exported method NoopLogger.Fatal should have comment or be unexported (golint)
    • Line 305: warning: exported method NoopLogger.Fatalf should have comment or be unexported (golint)
    • Line 308: warning: exported method NoopLogger.Fatalln should have comment or be unexported (golint)
    • Line 311: warning: exported method NoopLogger.Panic should have comment or be unexported (golint)
    • Line 314: warning: exported method NoopLogger.Panicf should have comment or be unexported (golint)
    • Line 317: warning: exported method NoopLogger.Panicln should have comment or be unexported (golint)
    • flytestdlib/profutils/server.go
    • Line 30: warning: exported type BuildVersion should have comment or be unexported (golint)
    • Line 36: warning: comment on exported function WriteStringResponse should be of the form "WriteStringResponse ..." (golint)
    • Line 43: warning: comment on exported function WriteJSONResponse should be of the form "WriteJSONResponse ..." (golint)
    • Line 94: warning: comment on exported function StartProfilingServer should be of the form "StartProfilingServer ..." (golint)
    • Line 122: warning: comment on exported function StartProfilingServerWithDefaultHandlers should be of the form "StartProfilingServerWithDefaultHandlers ..." (golint)
    • flytestdlib/utils/marshal_utils.go
    • Line 16: warning: comment on exported function UnmarshalStructToPb should be of the form "UnmarshalStructToPb ..." (golint)
    • Line 38: warning: comment on exported function MarshalPbToStruct should be of the form "MarshalPbToStruct ..." (golint)
    • Line 57: warning: comment on exported function MarshalPbToString should be of the form "MarshalPbToString ..." (golint)
    • Line 62: warning: comment on exported function MarshalObjToStruct should be of the form "MarshalObjToStruct ..." (golint)
    • flytestdlib/config/viper/collection.go
    • Line 20: warning: exported type Viper should have comment or be unexported (golint)
    • Line 32: warning: comment on exported type CollectionProxy should be of the form "CollectionProxy ..." (with optional leading article) (golint)
    • Line 40: warning: exported method CollectionProxy.BindPFlags should have comment or be unexported (golint)
    • Line 51: warning: exported method CollectionProxy.BindEnv should have comment or be unexported (golint)
    • Line 66: warning: exported method CollectionProxy.AutomaticEnv should have comment or be unexported (golint)
    • Line 74: warning: exported method CollectionProxy.ReadInConfig should have comment or be unexported (golint)
    • Line 83: warning: exported method CollectionProxy.OnConfigChange should have comment or be unexported (golint)
    • Line 89: warning: exported method CollectionProxy.WatchConfig should have comment or be unexported (golint)
    • Line 95: warning: exported method CollectionProxy.AllSettings should have comment or be unexported (golint)
    • Line 110: warning: exported method CollectionProxy.ConfigFileUsed should have comment or be unexported (golint)
    • Line 114: warning: exported method CollectionProxy.MergeConfig should have comment or be unexported (golint)
    • Line 118: warning: exported method CollectionProxy.MergeAllConfigs should have comment or be unexported (golint)
    • Line 165: warning: exported method CollectionProxy.ConfigFilesUsed should have comment or be unexported (golint)
    • flytestdlib/promutils/labeled/metric_option.go
    • Line 3: warning: comment on exported type MetricOption should be of the form "MetricOption ..." (with optional leading article) (golint)
    • Line 8: warning: comment on exported type EmitUnlabeledMetricOption should be of the form "EmitUnlabeledMetricOption ..." (with optional leading article) (golint)
    • Line 15: warning: exported var EmitUnlabeledMetric should have comment or be unexported (golint)
    • flytestdlib/config/url.go
    • Line 9: warning: comment on exported type URL should be of the form "URL ..." (with optional leading article) (golint)
    • Line 14: warning: exported method URL.MarshalJSON should have comment or be unexported (golint)
    • Line 18: warning: exported method URL.UnmarshalJSON should have comment or be unexported (golint)
    • flytestdlib/storage/protobuf_store.go
    • Line 29: warning: comment on exported type DefaultProtobufStore should be of the form "DefaultProtobufStore ..." (with optional leading article) (golint)
    • Line 35: warning: exported method DefaultProtobufStore.ReadProtobuf should have comment or be unexported (golint)
    • Line 66: warning: exported method DefaultProtobufStore.WriteProtobuf should have comment or be unexported (golint)
    • Line 84: warning: exported function NewDefaultProtobufStore should have comment or be unexported (golint)
    • flytestdlib/storage/storage.go
    • Line 1: warning: package comment should be of the form "Package storage ..." (golint)
    • Line 19: warning: comment on exported type DataReference should be of the form "DataReference ..." (with optional leading article) (golint)
    • Line 24: warning: comment on exported type Options should be of the form "Options ..." (with optional leading article) (golint)
    • Line 30: warning: comment on exported type Metadata should be of the form "Metadata ..." (with optional leading article) (golint)
    • Line 36: warning: comment on exported type DataStore should be of the form "DataStore ..." (with optional leading article) (golint)
    • Line 45: warning: comment on exported type RawStore should be of the form "RawStore ..." (with optional leading article) (golint)
    • Line 65: warning: comment on exported type ReferenceConstructor should be of the form "ReferenceConstructor ..." (with optional leading article) (golint)
    • Line 71: warning: comment on exported type ProtobufStore should be of the form "ProtobufStore ..." (with optional leading article) (golint)
    • Line 82: warning: comment on exported type ComposedProtobufStore should be of the form "ComposedProtobufStore ..." (with optional leading article) (golint)
    • Line 89: warning: comment on exported method DataReference.Split should be of the form "Split ..." (golint)
    • flytestdlib/errors/errors.go
    • Line 5: warning: comment on exported type ErrorCollection should be of the form "ErrorCollection ..." (with optional leading article) (golint)
    • Line 17: warning: exported method ErrorCollection.ErrorOrDefault should have comment or be unexported (golint)
    • Line 25: warning: exported method ErrorCollection.Append should have comment or be unexported (golint)
    • flytestdlib/config/accessor.go
    • Line 1: warning: package comment should be of the form "Package config ..." (golint)
    • Line 31: warning: comment on exported type Accessor should be of the form "Accessor ..." (with optional leading article) (golint)
    • flytestdlib/cli/pflags/api/generator.go
    • Line 42: warning: exported type SliceOrArray should have comment or be unexported (golint)
    • Line 432: warning: exported method PFlagProviderGenerator.GetTargetPackage should have comment or be unexported (golint)
    • Line 436: warning: exported method PFlagProviderGenerator.Generate should have comment or be unexported (golint)
    • flytestdlib/storage/utils.go
    • Line 15: warning: exported var ErrExceedsLimit should have comment or be unexported (golint)
    • Line 23: warning: comment on exported function IsNotFound should be of the form "IsNotFound ..." (golint)
    • Line 36: warning: comment on exported function IsExists should be of the form "IsExists ..." (golint)
    • Line 45: warning: comment on exported function IsExceedsLimit should be of the form "IsExceedsLimit ..." (golint)
    • Line 50: warning: exported function IsFailedWriteToCache should have comment or be unexported (golint)
    • Line 54: warning: exported function MapStrings should have comment or be unexported (golint)
    • flytestdlib/config/viper/viper.go
    • Line 394: warning: comment on exported function NewAccessor should be of the form "NewAccessor ..." (golint)
    • Line 428: warning: comment on exported function GetConfigCommand should be of the form "GetConfigCommand ..." (golint)
    • flytestdlib/config/port.go
    • Line 10: warning: comment on exported type Port should be of the form "Port ..." (with optional leading article) (golint)
    • Line 19: warning: exported method Port.MarshalJSON should have comment or be unexported (golint)
    • Line 23: warning: exported method Port.UnmarshalJSON should have comment or be unexported (golint)
    • flytestdlib/cli/pflags/api/sample.go
    • Line 10: warning: exported var DefaultTestType should have comment or be unexported (golint)
    • Line 14: warning: exported type TestType should have comment or be unexported (golint)
    • Line 28: warning: exported type NestedType should have comment or be unexported (golint)
    • Line 32: warning: exported type ComplexJSONType should have comment or be unexported (golint)
    • Line 37: warning: exported method ComplexJSONType.UnmarshalJSON should have comment or be unexported (golint)
    • flytestdlib/cli/pflags/api/tag.go
    • Line 12: warning: exported const TagName should have comment (or a comment on this block) or be unexported (golint)
    • Line 16: warning: comment on exported type Tag should be of the form "Tag ..." (with optional leading article) (golint)
    • Line 27: warning: comment on exported function ParseTag should be of the form "ParseTag ..." (golint)
    • flytestdlib/ioutils/bytes.go
    • Line 8: warning: comment on exported type BytesReadCloser should be of the form "BytesReadCloser ..." (with optional leading article) (golint)
    • Line 13: warning: exported method BytesReadCloser.Close should have comment or be unexported (golint)
    • Line 17: warning: exported function NewBytesReadCloser should have comment or be unexported (golint)
    • flytestdlib/promutils/labeled/counter.go
    • Line 11: warning: comment on exported type Counter should be of the form "Counter ..." (with optional leading article) (golint)
    • Line 48: warning: comment on exported function NewCounter should be of the form "NewCounter ..." (golint)
    • flytestdlib/storage/rawstores.go
    • Line 57: warning: comment on exported function NewDataStore should be of the form "NewDataStore ..." (golint)
    • Line 80: warning: comment on exported function NewCompositeDataStore should be of the form "NewCompositeDataStore ..." (golint)
    • flytestdlib/atomic/atomic.go
    • Line 35: warning: comment on exported method Bool.CompareAndSwap should be of the form "CompareAndSwap ..." (golint)
    • Line 55: warning: exported type Uint32 should have comment or be unexported (golint)
    • Line 59: warning: comment on exported method Uint32.Load should be of the form "Load ..." (golint)
    • Line 64: warning: comment on exported method Uint32.CompareAndSwap should be of the form "CompareAndSwap ..." (golint)
    • Line 74: warning: comment on exported method Uint32.Inc should be of the form "Inc ..." (golint)
    • Line 79: warning: comment on exported method Uint32.Store should be of the form "Store ..." (golint)
    • Line 84: warning: exported function NewUint32 should have comment or be unexported (golint)
    • Line 88: warning: exported type Int32 should have comment or be unexported (golint)
    • Line 92: warning: comment on exported method Int32.Load should be of the form "Load ..." (golint)
    • Line 97: warning: comment on exported method Int32.CompareAndSwap should be of the form "CompareAndSwap ..." (golint)
    • Line 107: warning: comment on exported method Int32.Sub should be of the form "Sub ..." (golint)
    • Line 112: warning: comment on exported method Int32.Inc should be of the form "Inc ..." (golint)
    • Line 117: warning: comment on exported method Int32.Dec should be of the form "Dec ..." (golint)
    • Line 122: warning: comment on exported method Int32.Store should be of the form "Store ..." (golint)
    • Line 127: warning: exported function NewInt32 should have comment or be unexported (golint)
    • flytestdlib/contextutils/context.go
    • Line 1: warning: package comment should be of the form "Package contextutils ..." (golint)
    • Line 10: warning: exported type Key should have comment or be unexported (golint)
    • Line 13: warning: exported const AppNameKey should have comment (or a comment on this block) or be unexported (golint)
    • Line 58: warning: comment on exported function WithResourceVersion should be of the form "WithResourceVersion ..." (golint)
    • Line 63: warning: comment on exported function WithNamespace should be of the form "WithNamespace ..." (golint)
    • Line 68: warning: comment on exported function WithJobID should be of the form "WithJobID ..." (golint)
    • Line 79: warning: comment on exported function WithAppName should be of the form "WithAppName ..." (golint)
    • Line 84: warning: comment on exported function WithPhase should be of the form "WithPhase ..." (golint)
    • Line 89: warning: comment on exported function WithExecutionID should be of the form "WithExecutionID ..." (golint)
    • Line 94: warning: comment on exported function WithNodeID should be of the form "WithNodeID ..." (golint)
    • Line 103: warning: comment on exported function WithWorkflowID should be of the form "WithWorkflowID ..." (golint)
    • Line 108: warning: comment on exported function WithLaunchPlanID should be of the form "WithLaunchPlanID ..." (golint)
    • Line 113: warning: comment on exported function WithProjectDomain should be of the form "WithProjectDomain ..." (golint)
    • Line 119: warning: comment on exported function WithTaskID should be of the form "WithTaskID ..." (golint)
    • Line 124: warning: comment on exported function WithTaskType should be of the form "WithTaskType ..." (golint)
    • Line 129: warning: comment on exported function WithGoroutineLabel should be of the form "WithGoroutineLabel ..." (golint)
    • Line 153: warning: comment on exported function GetLogFields should be of the form "GetLogFields ..." (golint)
    • Line 164: warning: exported function Value should have comment or be unexported (golint)
    • Line 173: warning: exported function Values should have comment or be unexported (golint)
    • flytestdlib/random/weighted_random_list.go
    • Line 15: warning: comment on exported type WeightedRandomList should be of the form "WeightedRandomList ..." (with optional leading article) (golint)
    • Line 23: warning: comment on exported type Comparable should be of the form "Comparable ..." (with optional leading article) (golint)
    • Line 28: warning: comment on exported type Entry should be of the form "Entry ..." (with optional leading article) (golint)
    • Line 62: warning: comment on exported function NewWeightedRandom should be of the form "NewWeightedRandom ..." (golint)
    • flytestdlib/atomic/non_blocking_lock.go
    • Line 3: warning: comment on exported type NonBlockingLock should be of the form "NonBlockingLock ..." (with optional leading article) (golint)
    • Line 9: warning: exported function NewNonBlockingLock should have comment or be unexported (golint)
    • flytestdlib/bitarray/compact_array.go
    • Line 1: warning: package comment should be of the form "Package bitarray ..." (golint)
    • Line 11: warning: comment on exported type Item should be of the form "Item ..." (with optional leading article) (golint)
    • Line 14: warning: comment on exported type CompactArray should be of the form "CompactArray ..." (with optional leading article) (golint)
    • Line 38: warning: comment on exported method CompactArray.SetItem should be of the form "SetItem ..." (golint)
    • Line 55: warning: comment on exported method CompactArray.GetItem should be of the form "GetItem ..." (golint)
    • Line 72: warning: comment on exported method CompactArray.GetItems should be of the form "GetItems ..." (golint)
    • Line 117: warning: comment on exported function NewCompactArray should be of the form "NewCompactArray ..." (golint)
    • flytestdlib/config/regexp.go
    • Line 9: warning: comment on exported type Regexp should be of the form "Regexp ..." (with optional leading article) (golint)
    • Line 14: warning: exported method Regexp.MarshalJSON should have comment or be unexported (golint)
    • Line 18: warning: exported method Regexp.UnmarshalJSON should have comment or be unexported (golint)
    • flytestdlib/cli/pflags/api/types.go
    • Line 8: warning: comment on exported type TestStrategy should be of the form "TestStrategy ..." (with optional leading article) (golint)
    • Line 12: warning: exported const JSON should have comment (or a comment on this block) or be unexported (golint)
    • Line 17: warning: exported type FieldInfo should have comment or be unexported (golint)
    • Line 30: warning: comment on exported type TypeInfo should be of the form "TypeInfo ..." (with optional leading article) (golint)
    • flytestdlib/bitarray/bitset.go
    • Line 10: warning: exported type Block should have comment or be unexported (golint)
    • Line 15: warning: comment on exported method BitSet.Set should be of the form "Set ..." (golint)
    • Line 24: warning: comment on exported method BitSet.Clear should be of the form "Clear ..." (golint)
    • Line 31: warning: comment on exported method BitSet.IsSet should be of the form "IsSet ..." (golint)
    • Line 40: warning: comment on exported method BitSet.BlockCount should be of the form "BlockCount ..." (golint)
    • Line 45: warning: comment on exported method BitSet.Cap should be of the form "Cap ..." (golint)
    • Line 68: warning: comment on exported function NewBitSet should be of the form "NewBitSet ..." (golint)
    • flytestdlib/promutils/labeled/gauge.go
    • Line 11: warning: comment on exported type Gauge should be of the form "Gauge ..." (with optional leading article) (golint)
    • Line 102: warning: comment on exported function NewGauge should be of the form "NewGauge ..." (golint)
    • flytestdlib/promutils/scope.go
    • Line 26: warning: comment on exported type StopWatch should be of the form "StopWatch ..." (with optional leading article) (golint)
    • Line 49: warning: comment on exported method StopWatch.Observe should be of the form "Observe ..." (golint)
    • Line 61: warning: comment on exported method StopWatch.Time should be of the form "Time ..." (golint)
    • Line 68: warning: comment on exported type StopWatchVec should be of the form "StopWatchVec ..." (with optional leading article) (golint)
    • Line 76: warning: comment on exported method StopWatchVec.WithLabelValues should be of the form "WithLabelValues ..." (golint)
    • Line 84: warning: exported method StopWatchVec.GetMetricWith should have comment or be unexported (golint)
    • Line 95: warning: comment on exported type Timer should be of the form "Timer ..." (with optional leading article) (golint)
    • Line 103: warning: comment on exported method Timer.Stop should be of the form "Stop ..." (golint)
    • Line 424: warning: comment on exported function NewScope should be of the form "NewScope ..." (golint)
    • Line 441: warning: comment on exported function NewTestScope should be of the form "NewTestScope ..." (golint)
    • flytestdlib/config/utils.go
    • Line 13: warning: comment on exported function DeepCopyConfig should be of the form "DeepCopyConfig ..." (golint)
    • Line 30: warning: exported function DeepEqual should have comment or be unexported (golint)
    • Line 45: warning: comment on exported function AllConfigsAsMap should be of the form "AllConfigsAsMap ..." (golint)
    • flytestdlib/pbhash/pbhash.go
    • Line 1: warning: package comment should be of the form "Package pbhash ..." (golint)
    • Line 24: warning: comment on exported function ComputeHash should be of the form "ComputeHash ..." (golint)
    • Line 50: warning: comment on exported function ComputeHashString should be of the form "ComputeHashString ..." (golint)
    • flytestdlib/config/duration.go
    • Line 9: warning: comment on exported type Duration should be of the form "Duration ..." (with optional leading article) (golint)
    • Line 14: warning: exported method Duration.MarshalJSON should have comment or be unexported (golint)
    • Line 18: warning: exported method Duration.UnmarshalJSON should have comment or be unexported (golint)
    • flytestdlib/storage/stow_store.go
    • Line 32: warning: exported const FailureTypeLabel should have comment (or a comment on this block) or be unexported (golint)
    • Line 93: warning: comment on exported type StowMetadata should be of the form "StowMetadata ..." (with optional leading article) (golint)
    • Line 99: warning: exported method StowMetadata.Size should have comment or be unexported (golint)
    • Line 103: warning: exported method StowMetadata.Exists should have comment or be unexported (golint)
    • Line 107: warning: comment on exported type StowStore should be of the form "StowStore ..." (with optional leading article) (golint)
    • Line 122: warning: exported method StowStore.CreateContainer should have comment or be unexported (golint)
    • Line 131: warning: exported method StowStore.LoadContainer should have comment or be unexported (golint)
    • Line 173: warning: exported method StowStore.Head should have comment or be unexported (golint)
    • Line 208: warning: exported method StowStore.ReadRaw should have comment or be unexported (golint)
    • Line 240: warning: exported method StowStore.WriteRaw should have comment or be unexported (golint)
    • Line 273: warning: exported method StowStore.GetBaseContainerFQN should have comment or be unexported (golint)
    • Line 277: warning: exported function NewStowRawStore should have comment or be unexported (golint)
    • flytestdlib/futures/future.go
    • Line 1: warning: package comment should be of the form "Package futures ..." (golint)
    • Line 17: warning: comment on exported type Future should be of the form "Future ..." (with optional leading article) (golint)
    • Line 26: warning: comment on exported type SyncFuture should be of the form "SyncFuture ..." (with optional leading article) (golint)
    • Line 35: warning: comment on exported method SyncFuture.Ready should be of the form "Ready ..." (golint)
    • Line 40: warning: comment on exported method SyncFuture.Get should be of the form "Get ..." (golint)
    • Line 45: warning: comment on exported function NewSyncFuture should be of the form "NewSyncFuture ..." (golint)
    • Line 56: warning: comment on exported type AsyncFuture should be of the form "AsyncFuture ..." (with optional leading article) (golint)
    • Line 83: warning: comment on exported method AsyncFuture.Ready should be of the form "Ready ..." (golint)
    • Line 90: warning: comment on exported method AsyncFuture.Get should be of the form "Get ..." (golint)
    • Line 102: warning: comment on exported function NewAsyncFuture should be of the form "NewAsyncFuture ..." (golint)
    • flytestdlib/config/section.go
    • Line 16: warning: exported type Section should have comment or be unexported (golint)
    • Line 60: warning: exported type Config should have comment or be unexported (golint)
    • Line 61: warning: exported type SectionKey should have comment or be unexported (golint)
    • Line 62: warning: exported type SectionMap should have comment or be unexported (golint)
    • Line 64: warning: comment on exported type PFlagProvider should be of the form "PFlagProvider ..." (with optional leading article) (golint)
    • Line 69: warning: exported type SectionUpdated should have comment or be unexported (golint)
    • Line 82: warning: comment on exported function GetRootSection should be of the form "GetRootSection ..." (golint)
    • Line 87: warning: exported function MustRegisterSection should have comment or be unexported (golint)
    • Line 105: warning: comment on exported function RegisterSection should be of the form "RegisterSection ..." (golint)
    • Line 116: warning: exported function MustRegisterSectionWithUpdates should have comment or be unexported (golint)
    • Line 134: warning: comment on exported function RegisterSectionWithUpdates should be of the form "RegisterSectionWithUpdates ..." (golint)
    • Line 169: warning: comment on exported function GetSection should be of the form "GetSection ..." (golint)
    • Line 222: warning: exported function NewSection should have comment or be unexported (golint)
    • Line 232: warning: exported function NewRootSection should have comment or be unexported (golint)
    • flytestdlib/logger/config.go
    • Line 13: warning: exported type FormatterType should have comment or be unexported (golint)
    • Line 16: warning: exported const FormatterJSON should have comment (or a comment on this block) or be unexported (golint)
    • Line 37: warning: comment on exported type Config should be of the form "Config ..." (with optional leading article) (golint)
    • Line 52: warning: exported type FormatterConfig should have comment or be unexported (golint)
    • Line 56: warning: comment on exported function SetConfig should be of the form "SetConfig ..." (golint)
    • Line 66: warning: exported function GetConfig should have comment or be unexported (golint)
    • flytestdlib/ioutils/timed_readers.go
    • Line 8: warning: comment on exported type Timer should be of the form "Timer ..." (with optional leading article) (golint)
    • Line 14: warning: exported function ReadAll should have comment or be unexported (golint)
    • flytestdlib/storage/config.go
    • Line 12: warning: comment on exported type Type should be of the form "Type ..." (with optional leading article) (golint)
    • Line 19: warning: exported const TypeMemory should have comment (or a comment on this block) or be unexported (golint)
    • Line 27: warning: exported const KiB should have comment (or a comment on this block) or be unexported (golint)
    • Line 32: warning: exported var ConfigSection should have comment or be unexported (golint)
    • Line 46: warning: comment on exported type Config should be of the form "Config ..." (with optional leading article) (golint)
    • Line 71: warning: comment on exported type ConnectionConfig should be of the form "ConnectionConfig ..." (with optional leading article) (golint)
    • Line 81: warning: exported type StowConfig should have comment or be unexported (golint)
    • Line 86: warning: exported type CachingConfig should have comment or be unexported (golint)
    • Line 101: warning: comment on exported type LimitsConfig should be of the form "LimitsConfig ..." (with optional leading article) (golint)
    • Line 106: warning: comment on exported function GetConfig should be of the form "GetConfig ..." (golint)
    • flytestdlib/promutils/labeled/keys.go
    • Line 12: warning: exported var ErrAlreadySet should have comment or be unexported (golint)
    • Line 25: warning: comment on exported function SetMetricKeys should be of the form "SetMetricKeys ..." (golint)
    • Line 45: warning: exported function GetUnlabeledMetricName should have comment or be unexported (golint)
    • Line 49: warning: comment on exported function UnsetMetricKeys should be of the form "UnsetMetricKeys ..." (golint)
    • flytestdlib/utils/auto_refresh_cache.go
    • Line 31: warning: comment on exported type CacheItem should be of the form "CacheItem ..." (with optional leading article) (golint)
    • Line 36: warning: comment on exported type CacheSyncAction should be of the form "CacheSyncAction ..." (with optional leading article) (golint)
    • Line 41: warning: exported const Unchanged should have comment (or a comment on this block) or be unexported (golint)
    • Line 43: warning: comment on exported const Update should be of the form "Update ..." (golint)
    • Line 46: warning: comment on exported const Delete should be of the form "Delete ..." (golint)
    • Line 50: warning: comment on exported type CacheSyncItem should be of the form "CacheSyncItem ..." (with optional leading article) (golint)
    • Line 64: warning: comment on exported function NewAutoRefreshCache should be of the form "NewAutoRefreshCache ..." (golint)
    • flytestdlib/storage/mem_store.go
    • Line 15: warning: exported type InMemoryStore should have comment or be unexported (golint)
    • Line 20: warning: exported type MemoryMetadata should have comment or be unexported (golint)
    • Line 25: warning: exported method MemoryMetadata.Size should have comment or be unexported (golint)
    • Line 29: warning: exported method MemoryMetadata.Exists should have comment or be unexported (golint)
    • Line 33: warning: exported method InMemoryStore.Head should have comment or be unexported (golint)
    • Line 38: warning: exported method InMemoryStore.ReadRaw should have comment or be unexported (golint)
    • Line 46: warning: exported method InMemoryStore.WriteRaw should have comment or be unexported (golint)
    • Line 58: warning: exported method InMemoryStore.Clear should have comment or be unexported (golint)
    • Line 63: warning: exported method InMemoryStore.GetBaseContainerFQN should have comment or be unexported (golint)
    • Line 67: warning: exported function NewInMemoryRawStore should have comment or be unexported (golint)
    • flytestdlib/storage/url_path.go
    • Line 17: warning: comment on exported type URLPathConstructor should be of the form "URLPathConstructor ..." (with optional leading article) (golint)
    • Line 29: warning: exported method URLPathConstructor.ConstructReference should have comment or be unexported (golint)
    • flytestdlib/errors/error.go
    • Line 1: warning: package comment should be of the form "Package errors ..." (golint)
    • Line 9: warning: comment on exported type ErrorCode should be of the form "ErrorCode ..." (with optional leading article) (golint)
    • Line 53: warning: comment on exported function Errorf should be of the form "Errorf ..." (golint)
    • Line 61: warning: comment on exported function Wrapf should be of the form "Wrapf ..." (golint)
    • Line 72: warning: comment on exported function GetErrorCode should be of the form "GetErrorCode ..." (golint)
    • Line 86: warning: comment on exported function IsCausedBy should be of the form "IsCausedBy ..." (golint)
    • Line 117: warning: exported function IsCausedByError should have comment or be unexported (golint)
    • flytestdlib/promutils/labeled/stopwatch.go
    • Line 11: warning: exported type StopWatch should have comment or be unexported (golint)
    • Line 48: warning: comment on exported method StopWatch.Observe should be of the form "Observe ..." (golint)
    • Line 62: warning: comment on exported method StopWatch.Time should be of the form "Time ..." (golint)
    • Line 71: warning: comment on exported function NewStopWatch should be of the form "NewStopWatch ..." (golint)
    • flytestdlib/cli/pflags/api/pflag_provider.go
    • Line 15: warning: exported type PFlagProvider should have comment or be unexported (golint)
    • Line 21: warning: comment on exported method PFlagProvider.Imports should be of the form "Imports ..." (golint)
    • Line 31: warning: comment on exported method PFlagProvider.WriteCodeFile should be of the form "WriteCodeFile ..." (golint)
    • Line 42: warning: comment on exported method PFlagProvider.WriteTestFile should be of the form "WriteTestFile ..." (golint)
    • flytestdlib/internal/utils/parsers.go
    • Line 8: warning: comment on exported function MustParseURL should be of the form "MustParseURL ..." (golint)
    • Line 18: warning: comment on exported function RefInt should be of the form "RefInt ..." (golint)
    • Line 23: warning: comment on exported function MustCompileRegexp should be of the form "MustCompileRegexp ..." (golint)
    • flytestdlib/sets/generic_set.go
    • Line 7: warning: exported type SetObject should have comment or be unexported (golint)
    • Line 11: warning: exported type Generic should have comment or be unexported (golint)
    • Line 13: warning: comment on exported function NewGeneric should be of the form "NewGeneric ..." (golint)
    • Line 139: warning: comment on exported method Generic.ListKeys should be of the form "ListKeys ..." (golint)
    • Line 160: warning: comment on exported method Generic.UnsortedListKeys should be of the form "UnsortedListKeys ..." (golint)
    • Line 178: warning: comment on exported method Generic.PopAny should be of the form "PopAny ..." (golint)
    • flytestdlib/utils/rate_limiter.go
    • Line 10: warning: comment on exported type RateLimiter should be of the form "RateLimiter ..." (with optional leading article) (golint)
    • Line 29: warning: comment on exported function NewRateLimiter should be of the form "NewRateLimiter ..." (golint)
    • flytestdlib/config/config_cmd.go
    • Line 13: warning: exported const PathFlag should have comment (or a comment on this block) or be unexported (golint)
    • Line 19: warning: exported type AccessorProvider should have comment or be unexported (golint)
    • Line 26: warning: exported function NewConfigCommand 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!