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.

• tasques/internal/infra/elasticsearch/task/recurring/service.go
  • Line 44: warning: cyclomatic complexity 17 of function (*EsService).Create() is high (> 15) (gocyclo)

• tasques/internal/infra/elasticsearch/leader/lock.go
  • Line 270: warning: cyclomatic complexity 26 of function (*EsLock).loop() is high (> 15) (gocyclo)

golint36%

Golint is a linter for Go source code.

• tasques/example/ciphers/server/persistence/repo.go
  • Line 9: warning: comment on exported type MessagesRepo should be of the form "MessagesRepo ..." (with optional leading article) (golint)
  • Line 15: warning: exported function NewMessagesRepo should have comment or be unexported (golint)

• tasques/worker/config/models.go
  • Line 14: warning: exported type TasquesWorker should have comment or be unexported (golint)
  • Line 22: warning: exported type WorkerId should have comment or be unexported (golint)
  • Line 24: warning: exported type TasquesServer should have comment or be unexported (golint)
  • Line 31: warning: exported type BasicAuthUser should have comment or be unexported (golint)
  • Line 36: warning: exported method TasquesServer.BuildClient should have comment or be unexported (golint)
  • Line 47: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 58: warning: exported method WorkerId.Set should have comment or be unexported (golint)
  • Line 61: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 67: warning: exported method WorkerId.Type should have comment or be unexported (golint)

• tasques/example/ciphers/worker-go/cmd/root.go
  • Line 69: warning: comment on exported function Execute should be of the form "Execute ..." (golint)

• tasques/internal/api/models/task/recurring/models.go
  • Line 12: warning: comment on exported type Duration should be of the form "Duration ..." (with optional leading article) (golint)
  • Line 16: warning: comment on exported type TaskDefinition should be of the form "TaskDefinition ..." (with optional leading article) (golint)
  • Line 37: warning: comment on exported type NewTask should be of the form "NewTask ..." (with optional leading article) (golint)
  • Line 56: warning: comment on exported type TaskUpdate should be of the form "TaskUpdate ..." (with optional leading article) (golint)
  • Line 72: warning: comment on exported type Task should be of the form "Task ..." (with optional leading article) (golint)
  • Line 93: warning: comment on exported method NewTask.ToDomainNewTask should be of the form "ToDomainNewTask ..." (golint)
  • Line 109: warning: exported function FromDomainTask should have comment or be unexported (golint)
  • Line 136: warning: exported method TaskDefinition.ToDomainTaskDefinition should have comment or be unexported (golint)
  • Line 167: warning: exported method Duration.UnmarshalJSON should have comment or be unexported (golint)
  • Line 171: warning: exported method Duration.MarshalJSON should have comment or be unexported (golint)

• tasques/internal/domain/task/service.go
  • Line 100: warning: exported type WrappingErr should have comment or be unexported (golint)
  • Line 116: warning: exported method NotFound.Id should have comment or be unexported (golint)
  • Line 120: warning: comment on exported type InvalidVersion should be of the form "InvalidVersion ..." (with optional leading article) (golint)
  • Line 129: warning: exported method InvalidVersion.Id should have comment or be unexported (golint)
  • Line 143: warning: exported method AlreadyExists.Id should have comment or be unexported (golint)
  • Line 147: warning: comment on exported type InvalidPersistedData should be of the form "InvalidPersistedData ..." (with optional leading article) (golint)
  • Line 156: warning: exported type NotOwnedByWorker should have comment or be unexported (golint)
  • Line 165: warning: exported method NotOwnedByWorker.Id should have comment or be unexported (golint)
  • Line 169: warning: exported type NotClaimed should have comment or be unexported (golint)
  • Line 178: warning: exported method NotClaimed.Id should have comment or be unexported (golint)
  • Line 182: warning: exported type ReportFromThePast should have comment or be unexported (golint)
  • Line 192: warning: exported method ReportFromThePast.Id should have comment or be unexported (golint)
  • Line 196: warning: exported type Unclaimable should have comment or be unexported (golint)
  • Line 205: warning: exported method Unclaimable.Id should have comment or be unexported (golint)

• tasques/internal/infra/server/routing/tasks/routes_handler_test.go
  • Line 419: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 439: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 451: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 468: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 490: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 512: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 534: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/example/ciphers/server/cmd/root.go
  • Line 40: warning: comment on exported function Execute should be of the form "Execute ..." (golint)

• tasques/internal/infra/apm/tracing/impl.go
  • Line 11: warning: comment on exported function NewTracer should be of the form "NewTracer ..." (golint)
  • Line 51: warning: exported type NoopTracer should have comment or be unexported (golint)
  • Line 53: warning: exported method NoopTracer.BackgroundTx should have comment or be unexported (golint)

• tasques/internal/infra/elasticsearch/common/client.go
  • Line 24: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/infra/server/routing/tasks/recurring/routes_handler.go
  • Line 18: warning: exported type RoutesHandler should have comment or be unexported (golint)
  • Line 22: warning: exported method RoutesHandler.RegisterRoutes should have comment or be unexported (golint)

• tasques/worker/work_loop.go
  • Line 21: warning: exported type TaskHandle should have comment or be unexported (golint)
  • Line 29: warning: comment on exported method TaskHandle.ReportIn should be of the form "ReportIn ..." (golint)
  • Line 43: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 51: warning: comment on exported method TaskHandle.Unclaim should be of the form "Unclaim ..." (golint)
  • Line 59: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 65: warning: exported method TaskHandle.IsUnclaimed should have comment or be unexported (golint)
  • Line 73: warning: exported type Success should have comment or be unexported (golint)
  • Line 75: warning: exported type Failure should have comment or be unexported (golint)
  • Line 81: warning: comment on exported type TaskHandler should be of the form "TaskHandler ..." (with optional leading article) (golint)
  • Line 88: warning: exported type QueuesToHandles should have comment or be unexported (golint)
  • Line 90: warning: comment on exported type WorkLoop should be of the form "WorkLoop ..." (with optional leading article) (golint)
  • Line 105: warning: exported function NewWorkLoop should have comment or be unexported (golint)
  • Line 118: warning: comment on exported method WorkLoop.Run should be of the form "Run ..." (golint)
  • Line 219: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 237: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/example/ciphers/worker-go/persistence/repo.go
  • Line 9: warning: comment on exported type MessagesRepo should be of the form "MessagesRepo ..." (with optional leading article) (golint)
  • Line 15: warning: exported function NewMessagesRepo should have comment or be unexported (golint)

• tasques/example/ciphers/worker-go/handlers/ciphers.go
  • Line 19: warning: exported type CipherWorker should have comment or be unexported (golint)
  • Line 24: warning: exported method CipherWorker.Handle should have comment or be unexported (golint)
  • Line 33: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 41: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 50: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 73: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 89: warning: exported function Rotate should have comment or be unexported (golint)
  • Line 112: warning: exported function Base64 should have comment or be unexported (golint)
  • Line 116: warning: exported function Md5 should have comment or be unexported (golint)

• tasques/internal/infra/cron/task/recurring/scheduler.go
  • Line 30: warning: comment on exported function NewScheduler should be of the form "NewScheduler ..." (golint)
  • Line 145: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)

• tasques/internal/domain/task/recurring/manager.go
  • Line 25: warning: exported const NOT_LEADER should have comment (or a comment on this block) or be unexported (golint)
  • Line 61: warning: comment on exported function NewManager should be of the form "NewManager ..." (golint)
  • Line 123: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 232: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)

• tasques/internal/domain/metadata/models.go
  • Line 1: warning: package comment should be of the form "Package metadata ..." (golint)
  • Line 8: warning: exported type CreatedAt should have comment or be unexported (golint)
  • Line 9: warning: exported type ModifiedAt should have comment or be unexported (golint)
  • Line 11: warning: exported type SeqNum should have comment or be unexported (golint)
  • Line 12: warning: exported type PrimaryTerm should have comment or be unexported (golint)
  • Line 14: warning: exported type Version should have comment or be unexported (golint)
  • Line 19: warning: exported type Metadata should have comment or be unexported (golint)

• tasques/internal/infra/elasticsearch/common/models.go
  • Line 1: warning: package comment should be of the form "Package common ..." (golint)
  • Line 14: warning: exported type IndexName should have comment or be unexported (golint)
  • Line 15: warning: exported type DocumentID should have comment or be unexported (golint)
  • Line 17: warning: exported type ElasticsearchErr should have comment or be unexported (golint)
  • Line 29: warning: exported type JsonSerdesErr should have comment or be unexported (golint)
  • Line 40: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 45: warning: exported function UnexpectedEsStatusError should have comment or be unexported (golint)
  • Line 54: warning: exported type EsCreateResponse should have comment or be unexported (golint)
  • Line 60: warning: exported method EsCreateResponse.Version should have comment or be unexported (golint)
  • Line 67: warning: exported type EsUpdateResponse should have comment or be unexported (golint)
  • Line 75: warning: exported method EsUpdateResponse.Version should have comment or be unexported (golint)
  • Line 82: warning: exported type PersistedMetadata should have comment or be unexported (golint)
  • Line 87: warning: exported type EsBulkResponse should have comment or be unexported (golint)
  • Line 93: warning: exported type EsBulkResponseItemInfo should have comment or be unexported (golint)
  • Line 102: warning: exported type EsBulkResponseItem should have comment or be unexported (golint)
  • Line 109: warning: exported method EsBulkResponseItem.Info should have comment or be unexported (golint)
  • Line 122: warning: exported method EsBulkResponseItemInfo.IsOk should have comment or be unexported (golint)
  • Line 126: warning: exported method EsBulkResponseItemInfo.Version should have comment or be unexported (golint)
  • Line 133: warning: exported type EsCountResponse should have comment or be unexported (golint)

• tasques/internal/domain/task/models.go
  • Line 18: warning: exported type JsonObj should have comment or be unexported (golint)
  • Line 23: warning: comment on exported function GenerateId should be of the form "GenerateId ..." (golint)
  • Line 31: warning: comment on exported function GenerateNewId should be of the form "GenerateNewId ..." (golint)
  • Line 36: warning: comment on exported type Kind should be of the form "Kind ..." (with optional leading article) (golint)
  • Line 39: warning: comment on exported type Args should be of the form "Args ..." (with optional leading article) (golint)
  • Line 42: warning: comment on exported type Context should be of the form "Context ..." (with optional leading article) (golint)
  • Line 45: warning: comment on exported type RunAt should be of the form "RunAt ..." (with optional leading article) (golint)
  • Line 48: warning: exported type ProcessingTimeout should have comment or be unexported (golint)
  • Line 53: warning: comment on exported type RetryTimes should be of the form "RetryTimes ..." (with optional leading article) (golint)
  • Line 56: warning: comment on exported type AttemptedTimes should be of the form "AttemptedTimes ..." (with optional leading article) (golint)
  • Line 59: warning: comment on exported type RemainingAttempts should be of the form "RemainingAttempts ..." (with optional leading article) (golint)
  • Line 62: warning: comment on exported type NewTask should be of the form "NewTask ..." (with optional leading article) (golint)
  • Line 79: warning: comment on exported type EnqueuedAt should be of the form "EnqueuedAt ..." (with optional leading article) (golint)
  • Line 109: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 138: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 271: warning: comment on exported method Task.IntoUnClaimed should be of the form "IntoUnClaimed ..." (golint)
  • Line 295: warning: exported type ClaimedAt should have comment or be unexported (golint)
  • Line 297: warning: exported type TimesOutAt should have comment or be unexported (golint)
  • Line 299: warning: exported type Failure should have comment or be unexported (golint)
  • Line 300: warning: exported type Success should have comment or be unexported (golint)
  • Line 302: warning: exported type ReportedData should have comment or be unexported (golint)
  • Line 303: warning: exported type ReportedAt should have comment or be unexported (golint)
  • Line 305: warning: comment on exported type NewReport should be of the form "NewReport ..." (with optional leading article) (golint)
  • Line 316: warning: exported type LastClaimed should have comment or be unexported (golint)
  • Line 324: warning: exported type CompletedAt should have comment or be unexported (golint)
  • Line 326: warning: exported type Result should have comment or be unexported (golint)
  • Line 333: warning: comment on exported type State should be of the form "State ..." (with optional leading article) (golint)
  • Line 338: warning: exported const QUEUED should have comment (or a comment on this block) or be unexported (golint)
  • Line 390: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 395: warning: comment on exported type RecurringTaskId should be of the form "RecurringTaskId ..." (with optional leading article) (golint)

• tasques/internal/domain/worker/models.go
  • Line 3: warning: exported type Id should have comment or be unexported (golint)

• tasques/example/ciphers/common/models.go
  • Line 16: warning: exported type NewMessage should have comment or be unexported (golint)
  • Line 20: warning: exported type Message should have comment or be unexported (golint)
  • Line 35: warning: exported var Rot1 should have comment or be unexported (golint)
  • Line 42: warning: exported type MessageArgs should have comment or be unexported (golint)
  • Line 46: warning: exported method MessageArgs.ToMap should have comment or be unexported (golint)
  • Line 53: warning: exported function FromInterface should have comment or be unexported (golint)
  • Line 66: warning: exported type MessageCreate should have comment or be unexported (golint)
  • Line 70: warning: exported type MessageReads should have comment or be unexported (golint)
  • Line 75: warning: exported type MessageUpdate should have comment or be unexported (golint)
  • Line 82: warning: exported var MessagesIndexName should have comment or be unexported (golint)
  • Line 88: warning: exported type EsMessageRepo should have comment or be unexported (golint)
  • Line 92: warning: exported method EsMessageRepo.Get should have comment or be unexported (golint)
  • Line 114: warning: exported method EsMessageRepo.Create should have comment or be unexported (golint)
  • Line 153: warning: exported method EsMessageRepo.List should have comment or be unexported (golint)
  • Line 172: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 198: warning: exported method EsMessageRepo.SetRot1 should have comment or be unexported (golint)
  • Line 202: warning: exported method EsMessageRepo.SetRot13 should have comment or be unexported (golint)
  • Line 206: warning: exported method EsMessageRepo.SetBase64 should have comment or be unexported (golint)
  • Line 210: warning: exported method EsMessageRepo.SetMd5 should have comment or be unexported (golint)

• tasques/internal/domain/task/recurring/models.go
  • Line 11: warning: comment on exported type TaskDefinition should be of the form "TaskDefinition ..." (with optional leading article) (golint)
  • Line 22: warning: comment on exported type ScheduleExpression should be of the form "ScheduleExpression ..." (with optional leading article) (golint)
  • Line 26: warning: comment on exported type LoadedAt should be of the form "LoadedAt ..." (with optional leading article) (golint)
  • Line 37: warning: comment on exported type IsDeleted should be of the form "IsDeleted ..." (with optional leading article) (golint)
  • Line 40: warning: comment on exported type NewTask should be of the form "NewTask ..." (with optional leading article) (golint)
  • Line 50: warning: comment on exported type Task should be of the form "Task ..." (with optional leading article) (golint)
  • Line 68: warning: exported method Task.UpdateSchedule should have comment or be unexported (golint)
  • Line 73: warning: exported method Task.UpdateTaskDefinition should have comment or be unexported (golint)
  • Line 78: warning: exported method Task.IntoDeleted should have comment or be unexported (golint)

• tasques/internal/domain/task/recurring/service.go
  • Line 10: warning: exported type Service should have comment or be unexported (golint)
  • Line 63: warning: exported type BulkUpdateOtherError should have comment or be unexported (golint)
  • Line 76: warning: exported type WrappingErr should have comment or be unexported (golint)
  • Line 91: warning: exported method AlreadyExists.Id should have comment or be unexported (golint)
  • Line 105: warning: exported method NotFound.Id should have comment or be unexported (golint)
  • Line 109: warning: comment on exported type InvalidVersion should be of the form "InvalidVersion ..." (with optional leading article) (golint)
  • Line 118: warning: exported method InvalidVersion.Id should have comment or be unexported (golint)
  • Line 122: warning: comment on exported type InvalidPersistedData should be of the form "InvalidPersistedData ..." (with optional leading article) (golint)

• tasques/internal/infra/elasticsearch/task/service.go
  • Line 26: warning: exported var TasquesQueuePrefix should have comment or be unexported (golint)
  • Line 27: warning: exported var TasquesArchiveIndex should have comment or be unexported (golint)
  • Line 29: warning: exported type EsService should have comment or be unexported (golint)
  • Line 40: warning: comment on exported method EsService.SetUTCGetter should be of the form "SetUTCGetter ..." (golint)
  • Line 53: warning: exported function NewService should have comment or be unexported (golint)
  • Line 65: warning: exported method EsService.Create should have comment or be unexported (golint)
  • Line 124: warning: exported method EsService.Get should have comment or be unexported (golint)
  • Line 150: warning: exported method EsService.Claim should have comment or be unexported (golint)
  • Line 166: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 192: warning: exported method EsService.ReportIn should have comment or be unexported (golint)
  • Line 209: warning: exported method EsService.MarkDone should have comment or be unexported (golint)
  • Line 215: warning: exported method EsService.MarkFailed should have comment or be unexported (golint)
  • Line 221: warning: exported method EsService.UnClaim should have comment or be unexported (golint)
  • Line 237: warning: exported method EsService.ReapTimedOutTasks should have comment or be unexported (golint)
  • Line 258: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 265: warning: exported method EsService.ArchiveOldTasks should have comment or be unexported (golint)
  • Line 282: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 378: warning: exported method EsService.RefreshAsNeeded should have comment or be unexported (golint)
  • Line 408: warning: exported method EsService.OutstandingTasksCount should have comment or be unexported (golint)
  • Line 479: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 571: warning: exported function BuildIndexName should have comment or be unexported (golint)
  • Line 634: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)
  • Line 733: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 742: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 772: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 789: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 819: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 856: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 898: warning: exported type PersistedTaskData should have comment or be unexported (golint)
  • Line 997: warning: exported function ToPersistedTask should have comment or be unexported (golint)

• tasques/internal/infra/server/routing/tasks/routes_handler.go
  • Line 26: warning: exported var WorkerIdHeaderKey should have comment or be unexported (golint)
  • Line 30: warning: exported type RoutesHandler should have comment or be unexported (golint)
  • Line 35: warning: exported method RoutesHandler.RegisterRoutes should have comment or be unexported (golint)
  • Line 297: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/infra/elasticsearch/leader/lock.go
  • Line 24: warning: exported var IndexName should have comment or be unexported (golint)
  • Line 36: warning: exported const CHECKER should have comment (or a comment on this block) or be unexported (golint)
  • Line 51: warning: comment on exported type EsLock should be of the form "EsLock ..." (with optional leading article) (golint)
  • Line 80: warning: comment on exported method EsLock.SetUTCGetter should be of the form "SetUTCGetter ..." (golint)
  • Line 110: warning: exported method EsLock.IsLeader should have comment or be unexported (golint)
  • Line 398: warning: exported method EsLock.Start should have comment or be unexported (golint)
  • Line 404: warning: exported method EsLock.Stop should have comment or be unexported (golint)
  • Line 410: warning: exported type NotFound should have comment or be unexported (golint)
  • Line 416: warning: exported type Conflict should have comment or be unexported (golint)

• tasques/internal/infra/kibana/saved_objects/importer.go
  • Line 1: warning: don't use an underscore in package name (golint)
  • Line 42: warning: exported type Importer should have comment or be unexported (golint)
  • Line 108: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 122: warning: exported type RequestPrepErr should have comment or be unexported (golint)
  • Line 130: warning: exported type KibanaResponseErr should have comment or be unexported (golint)

• tasques/example/ciphers/common/config/models.go
  • Line 9: warning: exported type ElasticsearchClient should have comment or be unexported (golint)
  • Line 14: warning: exported method ElasticsearchClient.BuildClient should have comment or be unexported (golint)

• tasques/internal/infra/elasticsearch/index/lifecycle_management.go
  • Line 19: warning: exported var DefaultArchivedTasquesPolicyName should have comment or be unexported (golint)
  • Line 46: warning: exported type ILMSetup should have comment or be unexported (golint)
  • Line 67: warning: exported function NewILMSetup should have comment or be unexported (golint)
  • Line 147: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 156: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 191: warning: exported type PolicyNotInstalled should have comment or be unexported (golint)

• tasques/internal/domain/task/recurring/manager_test.go
  • Line 389: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 398: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 407: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/domain/task/recurring/mock_service.go
  • Line 11: warning: exported var MockNow should have comment or be unexported (golint)
  • Line 13: warning: exported type MockRecurringTasksService should have comment or be unexported (golint)
  • Line 30: warning: exported var MockDomainRecurringTask should have comment or be unexported (golint)
  • Line 49: warning: exported method MockRecurringTasksService.Create should have comment or be unexported (golint)
  • Line 53: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 58: warning: exported method MockRecurringTasksService.Get should have comment or be unexported (golint)
  • Line 62: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 67: warning: exported method MockRecurringTasksService.Delete should have comment or be unexported (golint)
  • Line 71: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 76: warning: exported method MockRecurringTasksService.All should have comment or be unexported (golint)
  • Line 80: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 85: warning: exported method MockRecurringTasksService.Update should have comment or be unexported (golint)
  • Line 89: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 94: warning: exported method MockRecurringTasksService.MarkLoaded should have comment or be unexported (golint)
  • Line 98: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 103: warning: exported method MockRecurringTasksService.NotLoaded should have comment or be unexported (golint)
  • Line 107: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/domain/leader/lock.go
  • Line 3: warning: exported type Checker should have comment or be unexported (golint)

• tasques/internal/infra/server/binding/validation/fields.go
  • Line 14: warning: exported function SetUpValidators should have comment or be unexported (golint)
  • Line 28: warning: exported var QueueNameValidatorTag should have comment or be unexported (golint)
  • Line 29: warning: exported var QueueNameValidator should have comment or be unexported (golint)
  • Line 40: warning: exported var ScheduleExpressionValidatorTag should have comment or be unexported (golint)
  • Line 55: warning: comment on exported type TestStandardParser should be of the form "TestStandardParser ..." (with optional leading article) (golint)
  • Line 58: warning: exported method TestStandardParser.Parse should have comment or be unexported (golint)

• tasques/internal/domain/task/recurring/scheduler.go
  • Line 9: warning: exported type Schedule should have comment or be unexported (golint)
  • Line 13: warning: exported type ScheduleParser should have comment or be unexported (golint)
  • Line 17: warning: comment on exported type Scheduler should be of the form "Scheduler ..." (with optional leading article) (golint)

• tasques/internal/domain/task/mock_service.go
  • Line 11: warning: exported var MockDomainTask should have comment or be unexported (golint)
  • Line 16: warning: exported type MockTasksService should have comment or be unexported (golint)
  • Line 41: warning: exported method MockTasksService.Create should have comment or be unexported (golint)
  • Line 45: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 50: warning: exported method MockTasksService.Get should have comment or be unexported (golint)
  • Line 54: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 59: warning: exported method MockTasksService.Claim should have comment or be unexported (golint)
  • Line 63: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 68: warning: exported method MockTasksService.ReportIn should have comment or be unexported (golint)
  • Line 72: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 77: warning: exported method MockTasksService.MarkDone should have comment or be unexported (golint)
  • Line 81: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 86: warning: exported method MockTasksService.MarkFailed should have comment or be unexported (golint)
  • Line 90: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 95: warning: exported method MockTasksService.UnClaim should have comment or be unexported (golint)
  • Line 99: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 104: warning: exported method MockTasksService.ReapTimedOutTasks should have comment or be unexported (golint)
  • Line 108: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 113: warning: exported method MockTasksService.ArchiveOldTasks should have comment or be unexported (golint)
  • Line 117: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 122: warning: exported method MockTasksService.RefreshAsNeeded should have comment or be unexported (golint)
  • Line 126: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 131: warning: exported method MockTasksService.OutstandingTasksCount should have comment or be unexported (golint)
  • Line 135: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/infra/elasticsearch/task/recurring/service.go
  • Line 20: warning: exported var TasquesRecurringTasksIndex should have comment or be unexported (golint)
  • Line 22: warning: exported type EsService should have comment or be unexported (golint)
  • Line 29: warning: exported method EsService.SetUTCGetter should have comment or be unexported (golint)
  • Line 33: warning: exported function NewService should have comment or be unexported (golint)
  • Line 44: warning: exported method EsService.Create should have comment or be unexported (golint)
  • Line 104: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 113: warning: exported method EsService.Get should have comment or be unexported (golint)
  • Line 146: warning: exported method EsService.Delete should have comment or be unexported (golint)
  • Line 150: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 155: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 161: warning: exported method EsService.All should have comment or be unexported (golint)
  • Line 174: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 179: warning: exported method EsService.NotLoaded should have comment or be unexported (golint)
  • Line 188: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 193: warning: exported method EsService.Update should have comment or be unexported (golint)
  • Line 232: warning: exported method EsService.MarkLoaded should have comment or be unexported (golint)
  • Line 398: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 504: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/app/cmd/root.go
  • Line 44: warning: comment on exported function Execute should be of the form "Execute ..." (golint)

• tasques/example/ciphers/server/html/templates.go
  • Line 4: warning: exported var Index should have comment or be unexported (golint)

• tasques/internal/config/models.go
  • Line 5: warning: exported type Tasques should have comment or be unexported (golint)
  • Line 9: warning: exported type TopLevel should have comment or be unexported (golint)
  • Line 13: warning: exported type App should have comment or be unexported (golint)
  • Line 26: warning: exported type Logging should have comment or be unexported (golint)
  • Line 32: warning: exported type KibanaClient should have comment or be unexported (golint)
  • Line 37: warning: exported type ElasticsearchClient should have comment or be unexported (golint)
  • Line 42: warning: exported type ApmClient should have comment or be unexported (golint)
  • Line 47: warning: exported type Tasks should have comment or be unexported (golint)
  • Line 52: warning: exported type Queues should have comment or be unexported (golint)
  • Line 58: warning: exported type TasksDefaults should have comment or be unexported (golint)
  • Line 69: warning: exported type TimedOutTasksReaper should have comment or be unexported (golint)
  • Line 75: warning: exported type TasksArchiver should have comment or be unexported (golint)
  • Line 82: warning: exported type RecurringTasks should have comment or be unexported (golint)
  • Line 89: warning: exported type Auth should have comment or be unexported (golint)
  • Line 93: warning: exported type BasicAuthUser should have comment or be unexported (golint)
  • Line 98: warning: exported type Recurring should have comment or be unexported (golint)
  • Line 105: warning: exported type LeaderLock should have comment or be unexported (golint)
  • Line 110: warning: exported type LifecycleSetup should have comment or be unexported (golint)
  • Line 114: warning: exported type LifecycleSettings should have comment or be unexported (golint)
  • Line 119: warning: exported type CustomLifecyclePolicy should have comment or be unexported (golint)

• tasques/internal/domain/queue/models.go
  • Line 52: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 60: warning: exported type InvalidName should have comment or be unexported (golint)

• tasques/internal/infra/elasticsearch/index/template.go
  • Line 19: warning: exported type TemplateName should have comment or be unexported (golint)
  • Line 20: warning: exported type Pattern should have comment or be unexported (golint)
  • Line 21: warning: exported type Json should have comment or be unexported (golint)
  • Line 22: warning: exported type Mappings should have comment or be unexported (golint)
  • Line 23: warning: exported type Settings should have comment or be unexported (golint)
  • Line 33: warning: exported method Template.Name should have comment or be unexported (golint)
  • Line 37: warning: exported function NewTemplate should have comment or be unexported (golint)
  • Line 41: warning: exported type TemplateSetup should have comment or be unexported (golint)
  • Line 58: warning: comment on exported function DefaultTemplateSetup should be of the form "DefaultTemplateSetup ..." (golint)
  • Line 81: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 113: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 146: warning: exported type PutTemplateErrors should have comment or be unexported (golint)
  • Line 154: warning: exported type TemplatesNotInstalled should have comment or be unexported (golint)
  • Line 164: warning: comment on exported var TasquesQueuesTemplate should be of the form "TasquesQueuesTemplate ..." (golint)
  • Line 172: warning: comment on exported var TasquesArchiveTemplate should be of the form "TasquesArchiveTemplate ..." (golint)
  • Line 180: warning: comment on exported var TasquesLocksTemplate should be of the form "TasquesLocksTemplate ..." (golint)
  • Line 192: warning: comment on exported var TasquesRecurringTasksTemplate should be of the form "TasquesRecurringTasksTemplate ..." (golint)

• tasques/internal/api/models/common/error.go
  • Line 3: warning: exported type Headers should have comment or be unexported (golint)
  • Line 11: warning: exported type ApiError should have comment or be unexported (golint)

• tasques/internal/infra/server/routing/global.go
  • Line 26: warning: exported function NewTopLevelRoutesGroup should have comment or be unexported (golint)
  • Line 45: warning: exported function NoRoute should have comment or be unexported (golint)
  • Line 49: warning: exported function NoMethod should have comment or be unexported (golint)
  • Line 53: warning: exported function HandleApiErr should have comment or be unexported (golint)
  • Line 57: warning: exported function HandleJsonSerdesErr should have comment or be unexported (golint)

• tasques/example/ciphers/worker-go/config/models.go
  • Line 8: warning: exported type App should have comment or be unexported (golint)
  • Line 13: warning: exported type CipherWorker should have comment or be unexported (golint)

• tasques/internal/api/controllers/task/controller.go
  • Line 45: warning: exported function New should have comment or be unexported (golint)
  • Line 66: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 76: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 86: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 100: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 110: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 120: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 130: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/infra/server/routing/tasks/recurring/routes_handler_test.go
  • Line 336: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 345: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 354: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 363: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 372: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/api/models/common/duration.go
  • Line 9: warning: exported type Duration should have comment or be unexported (golint)
  • Line 11: warning: exported method Duration.UnmarshalJSON should have comment or be unexported (golint)
  • Line 29: warning: exported method Duration.MarshalJSON should have comment or be unexported (golint)

• tasques/internal/api/controllers/task/recurring/controller.go
  • Line 15: warning: exported type Controller should have comment or be unexported (golint)
  • Line 38: warning: exported function New should have comment or be unexported (golint)
  • Line 50: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 60: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 74: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 85: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 95: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 105: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/example/ciphers/server/routing/server.go
  • Line 20: warning: exported type Server should have comment or be unexported (golint)
  • Line 24: warning: exported method Server.Run should have comment or be unexported (golint)
  • Line 55: warning: exported type Controller should have comment or be unexported (golint)
  • Line 141: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 146: warning: exported type IndexView should have comment or be unexported (golint)

• tasques/internal/infra/server/components.go
  • Line 44: warning: exported type Components should have comment or be unexported (golint)
  • Line 57: warning: exported function NewComponents should have comment or be unexported (golint)
  • Line 64: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
  • Line 109: warning: exported method Components.Run should have comment or be unexported (golint)
  • Line 229: warning: if block ends with a return statement, so drop this else and outdent its block (golint)

• tasques/internal/domain/tracing/interfaces.go
  • Line 5: warning: exported type Transaction should have comment or be unexported (golint)
  • Line 10: warning: exported type Tracer should have comment or be unexported (golint)

• tasques/internal/api/models/common/metadata.go
  • Line 9: warning: comment on exported type Version should be of the form "Version ..." (with optional leading article) (golint)
  • Line 25: warning: exported function FromDomainMetadata should have comment or be unexported (golint)

• tasques/internal/api/models/task/models.go
  • Line 1: warning: package comment should be of the form "Package task ..." (golint)
  • Line 15: warning: comment on exported type Duration should be of the form "Duration ..." (with optional leading article) (golint)
  • Line 45: warning: comment on exported type NewReport should be of the form "NewReport ..." (with optional leading article) (golint)
  • Line 69: warning: comment on exported type Claim should be of the form "Claim ..." (with optional leading article) (golint)
  • Line 80: warning: comment on exported type Success should be of the form "Success ..." (with optional leading article) (golint)
  • Line 85: warning: comment on exported type Failure should be of the form "Failure ..." (with optional leading article) (golint)
  • Line 90: warning: comment on exported type LastClaimed should be of the form "LastClaimed ..." (with optional leading article) (golint)
  • Line 104: warning: comment on exported type Task should be of the form "Task ..." (with optional leading article) (golint)
  • Line 138: warning: exported var TimeZero should have comment or be unexported (golint)
  • Line 140: warning: comment on exported method NewTask.ToDomainNewTask should be of the form "ToDomainNewTask ..." (golint)
  • Line 180: warning: comment on exported function FromDomainTask should be of the form "FromDomainTask ..." (golint)
  • Line 226: warning: exported method Duration.UnmarshalJSON should have comment or be unexported (golint)
  • Line 230: warning: exported method Duration.MarshalJSON should have comment or be unexported (golint)

• tasques/example/ciphers/server/config/models.go
  • Line 8: warning: exported type App should have comment or be unexported (golint)
  • Line 14: warning: exported type CipherQueuer should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!

ineffassign97%

IneffAssign detects ineffectual assignments in Go code.

• tasques/internal/infra/cron/task/recurring/scheduler_test.go
  • Line 213: warning: ineffectual assignment to err (ineffassign)

• tasques/internal/infra/elasticsearch/task/service.go
  • Line 203: warning: ineffectual assignment to timesRetried (ineffassign)
  • Line 231: warning: ineffectual assignment to timesRetried (ineffassign)
  • Line 506: warning: ineffectual assignment to timesRetried (ineffassign)

misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!