Preparing report...

Report for github.com/velocity-ci/velocity

A+    Excellent!    Found 41 issues across 66 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!


gocyclo96%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.


golint39%

Golint is a linter for Go source code.

    • velocity/backend/pkg/velocity/docker/compose/v3/docker-compose.go
    • Line 10: warning: exported type DockerComposeYaml should have comment or be unexported (golint)
    • Line 28: warning: exported type DockerComposeServiceCommand should have comment or be unexported (golint)
    • Line 30: warning: exported method DockerComposeServiceCommand.UnmarshalJSON should have comment or be unexported (golint)
    • Line 62: warning: exported type DockerComposeServiceEnvironment should have comment or be unexported (golint)
    • Line 64: warning: exported method DockerComposeServiceEnvironment.UnmarshalJSON should have comment or be unexported (golint)
    • Line 95: warning: exported type DockerComposeServiceNetwork should have comment or be unexported (golint)
    • Line 99: warning: exported type DockerComposeServiceBuild should have comment or be unexported (golint)
    • Line 104: warning: exported method DockerComposeServiceBuild.UnmarshalJSON should have comment or be unexported (golint)
    • Line 127: warning: exported type DockerComposeService should have comment or be unexported (golint)
    • Line 139: warning: exported function GetServiceOrder should have comment or be unexported (golint)
    • velocity/backend/pkg/git/status.go
    • Line 9: warning: exported method RawRepository.GetDescribe should have comment or be unexported (golint)
    • Line 18: warning: exported method RawRepository.GetDescribeAll should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/construction-plan.go
    • Line 28: warning: exported function NewConstructionPlanFromBlueprint should have comment or be unexported (golint)
    • Line 65: warning: exported function NewConstructionPlanFromPipeline should have comment or be unexported (golint)
    • Line 114: warning: exported method ConstructionPlan.Execute should have comment or be unexported (golint)
    • Line 138: warning: exported method ConstructionPlan.Stop should have comment or be unexported (golint)
    • velocity/backend/pkg/auth/jwt.go
    • Line 10: warning: exported var JWTSigningMethod should have comment or be unexported (golint)
    • Line 11: warning: exported var JWTStandardClaims should have comment or be unexported (golint)
    • Line 21: warning: exported function NewJWT should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/config/pipeline.go
    • Line 16: warning: exported type Stage should have comment or be unexported (golint)
    • Line 21: warning: exported method Stage.UnmarshalJSON should have comment or be unexported (golint)
    • Line 45: warning: exported type Pipeline should have comment or be unexported (golint)
    • Line 73: warning: exported method Pipeline.UnmarshalJSON should have comment or be unexported (golint)
    • Line 129: warning: exported function GetPipelinesFromRoot should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/step-setup.go
    • Line 21: warning: exported type Setup should have comment or be unexported (golint)
    • Line 50: warning: exported function NewStepSetup should have comment or be unexported (golint)
    • Line 65: warning: exported method Setup.GetDetails should have comment or be unexported (golint)
    • Line 85: warning: exported method Setup.Execute should have comment or be unexported (golint)
    • Line 179: warning: exported method Setup.Stop should have comment or be unexported (golint)
    • Line 183: warning: exported method Setup.SetParams should have comment or be unexported (golint)
    • Line 187: warning: exported method Setup.Validate should have comment or be unexported (golint)
    • Line 191: warning: exported function GetGlobalParams should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/config/parameter.go
    • Line 5: warning: exported type Parameter should have comment or be unexported (golint)
    • Line 8: warning: exported type BaseParameter should have comment or be unexported (golint)
    • Line 12: warning: exported function NewParameterBasic should have comment or be unexported (golint)
    • Line 20: warning: exported type ParameterBasic should have comment or be unexported (golint)
    • Line 28: warning: exported function NewParameterDerived should have comment or be unexported (golint)
    • Line 36: warning: exported type ParameterDerived should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/step.go
    • Line 11: warning: exported type Step should have comment or be unexported (golint)
    • Line 51: warning: exported type Stream should have comment or be unexported (golint)
    • Line 58: warning: exported type BaseStep should have comment or be unexported (golint)
    • Line 87: warning: exported method BaseStep.GetID should have comment or be unexported (golint)
    • Line 91: warning: exported method BaseStep.GetType should have comment or be unexported (golint)
    • Line 95: warning: exported method BaseStep.GetDescription should have comment or be unexported (golint)
    • Line 99: warning: exported method BaseStep.GetOutputStreams should have comment or be unexported (golint)
    • Line 103: warning: exported method BaseStep.GetStreamWriter should have comment or be unexported (golint)
    • Line 112: warning: exported method BaseStep.GetStreamID should have comment or be unexported (golint)
    • Line 121: warning: exported type StreamLine should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/step-docker-build.go
    • Line 15: warning: exported type StepDockerBuild should have comment or be unexported (golint)
    • Line 24: warning: exported function NewStepDockerBuild should have comment or be unexported (golint)
    • Line 33: warning: exported method StepDockerBuild.GetDetails should have comment or be unexported (golint)
    • Line 47: warning: exported method StepDockerBuild.Execute should have comment or be unexported (golint)
    • Line 84: warning: exported method StepDockerBuild.Stop should have comment or be unexported (golint)
    • Line 91: warning: exported method StepDockerBuild.Validate should have comment or be unexported (golint)
    • Line 95: warning: exported method StepDockerBuild.SetParams should have comment or be unexported (golint)
    • velocity/backend/pkg/git/repository.go
    • Line 20: warning: exported type Repository should have comment or be unexported (golint)
    • Line 27: warning: exported type RawRepository should have comment or be unexported (golint)
    • Line 33: warning: exported type CloneOptions should have comment or be unexported (golint)
    • Line 40: warning: exported type SSHKeyError should have comment or be unexported (golint)
    • Line 46: warning: exported type HostKeyError should have comment or be unexported (golint)
    • Line 120: warning: exported function Clone should have comment or be unexported (golint)
    • Line 172: warning: exported method RawRepository.Clean should have comment or be unexported (golint)
    • Line 182: warning: exported method RawRepository.Checkout should have comment or be unexported (golint)
    • Line 202: warning: exported method RawRepository.IsDirty should have comment or be unexported (golint)
    • velocity/backend/pkg/builder/task.go
    • Line 19: warning: exported const WorkspaceDir should have comment or be unexported (golint)
    • Line 26: warning: exported type ArchitectProject should have comment or be unexported (golint)
    • Line 32: warning: exported type KnownHost should have comment or be unexported (golint)
    • Line 36: warning: exported type Task should have comment or be unexported (golint)
    • Line 48: warning: exported function NewTask should have comment or be unexported (golint)
    • Line 56: warning: exported method Task.Parse should have comment or be unexported (golint)
    • Line 60: warning: exported method Task.Do should have comment or be unexported (golint)
    • Line 84: warning: exported method Task.Stop should have comment or be unexported (golint)
    • Line 89: warning: exported type BuildLogLine should have comment or be unexported (golint)
    • Line 96: warning: exported type TaskUpdatePayload should have comment or be unexported (golint)
    • Line 101: warning: exported type StepUpdatePayload should have comment or be unexported (golint)
    • Line 106: warning: exported type StreamNewLogLinePayload should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/docker-registries.go
    • Line 19: warning: exported type TaskDocker should have comment or be unexported (golint)
    • Line 23: warning: exported type DockerRegistry should have comment or be unexported (golint)
    • Line 98: warning: exported function GetAuthConfigsMap should have comment or be unexported (golint)
    • Line 118: warning: exported function GetAddressAuthTokensMap should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/step-docker-push.go
    • Line 16: warning: exported type StepDockerPush should have comment or be unexported (golint)
    • Line 23: warning: exported function NewStepDockerPush should have comment or be unexported (golint)
    • Line 30: warning: exported method StepDockerPush.GetDetails should have comment or be unexported (golint)
    • Line 40: warning: exported method StepDockerPush.Execute should have comment or be unexported (golint)
    • Line 72: warning: exported method StepDockerPush.Stop should have comment or be unexported (golint)
    • Line 79: warning: exported method StepDockerPush.Validate should have comment or be unexported (golint)
    • Line 83: warning: exported method StepDockerPush.SetParams should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/output/output.go
    • Line 10: warning: exported var ANSISuccess should have comment or be unexported (golint)
    • Line 23: warning: exported function ColorDisable should have comment or be unexported (golint)
    • Line 27: warning: exported function Italic should have comment or be unexported (golint)
    • Line 31: warning: exported function ColorFmt should have comment or be unexported (golint)
    • velocity/backend/pkg/git/commit.go
    • Line 15: warning: exported type RawCommit should have comment or be unexported (golint)
    • Line 24: warning: exported method RawRepository.GetCommitInfo should have comment or be unexported (golint)
    • Line 47: warning: exported method RawRepository.GetCurrentCommitInfo should have comment or be unexported (golint)
    • Line 58: warning: exported method RawRepository.GetTotalCommits should have comment or be unexported (golint)
    • velocity/backend/pkg/builder/parameter-resolver.go
    • Line 8: warning: exported type ParameterResolver should have comment or be unexported (golint)
    • Line 12: warning: exported function NewParameterResolver should have comment or be unexported (golint)
    • Line 18: warning: exported method ParameterResolver.Resolve should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/task.go
    • Line 16: warning: exported type Task should have comment or be unexported (golint)
    • Line 33: warning: exported method Task.UnmarshalJSON should have comment or be unexported (golint)
    • Line 123: warning: exported method Task.Execute should have comment or be unexported (golint)
    • Line 142: warning: exported method Task.Stop should have comment or be unexported (golint)
    • Line 168: warning: exported function NewTask should have comment or be unexported (golint)
    • Line 218: warning: exported method Task.UpdateSetup should have comment or be unexported (golint)
    • velocity/backend/pkg/git/branch.go
    • Line 10: warning: exported function CurrentBranch should have comment or be unexported (golint)
    • Line 25: warning: exported method RawRepository.GetDefaultBranch should have comment or be unexported (golint)
    • velocity/backend/pkg/phoenix/client.go
    • Line 21: warning: exported type Client should have comment or be unexported (golint)
    • Line 33: warning: exported function NewClient should have comment or be unexported (golint)
    • Line 47: warning: exported method Client.Wait should have comment or be unexported (golint)
    • Line 79: warning: exported method Client.Subscribe should have comment or be unexported (golint)
    • velocity/backend/pkg/phoenix/socket.go
    • Line 15: warning: exported function NewSocket should have comment or be unexported (golint)
    • Line 43: warning: exported type Socket should have comment or be unexported (golint)
    • Line 62: warning: exported method Socket.IsConnected should have comment or be unexported (golint)
    • Line 135: warning: exported method Socket.Send should have comment or be unexported (golint)
    • Line 217: warning: exported method Socket.ReplyOK should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/step-docker-run.go
    • Line 15: warning: exported type StepDockerRun should have comment or be unexported (golint)
    • Line 27: warning: exported function NewStepDockerRun should have comment or be unexported (golint)
    • Line 45: warning: exported method StepDockerRun.GetDetails should have comment or be unexported (golint)
    • Line 65: warning: exported method StepDockerRun.Execute should have comment or be unexported (golint)
    • Line 133: warning: exported method StepDockerRun.Stop should have comment or be unexported (golint)
    • Line 140: warning: exported method StepDockerRun.Validate should have comment or be unexported (golint)
    • Line 171: warning: exported method StepDockerRun.SetParams should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/writer.go
    • Line 3: warning: exported type StreamWriter should have comment or be unexported (golint)
    • Line 9: warning: exported type StepWriter should have comment or be unexported (golint)
    • Line 15: warning: exported type TaskWriter should have comment or be unexported (golint)
    • Line 28: warning: exported type BlankEmitter should have comment or be unexported (golint)
    • Line 31: warning: exported function NewBlankEmitter should have comment or be unexported (golint)
    • Line 35: warning: exported method BlankEmitter.GetStreamWriter should have comment or be unexported (golint)
    • Line 39: warning: exported method BlankEmitter.GetStepWriter should have comment or be unexported (golint)
    • Line 43: warning: exported method BlankEmitter.GetTaskWriter should have comment or be unexported (golint)
    • Line 47: warning: exported type BlankWriter should have comment or be unexported (golint)
    • Line 54: warning: exported method BlankWriter.SetStatus should have comment or be unexported (golint)
    • Line 56: warning: exported method BlankWriter.Close should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/config/root.go
    • Line 15: warning: exported type Root should have comment or be unexported (golint)
    • Line 24: warning: exported type RootProject should have comment or be unexported (golint)
    • Line 29: warning: exported type RootGit should have comment or be unexported (golint)
    • Line 34: warning: exported type RootPlugin should have comment or be unexported (golint)
    • Line 53: warning: exported method Root.UnmarshalJSON should have comment or be unexported (golint)
    • Line 107: warning: exported function GetRootConfig should have comment or be unexported (golint)
    • velocity/backend/pkg/phoenix/message.go
    • Line 24: warning: exported type PhoenixMessage should have comment or be unexported (golint)
    • Line 31: warning: exported type PhoenixReplyPayload should have comment or be unexported (golint)
    • Line 36: warning: exported type PhoenixGuardianJoinPayload should have comment or be unexported (golint)
    • Line 40: warning: exported method PhoenixMessage.UnmarshalJSON should have comment or be unexported (golint)
    • velocity/backend/pkg/builder/builder.go
    • Line 22: warning: exported const PoolTopic should have comment (or a comment on this block) or be unexported (golint)
    • Line 25: warning: exported type Builder should have comment or be unexported (golint)
    • Line 38: warning: exported method Builder.Stop should have comment or be unexported (golint)
    • Line 42: warning: exported method Builder.Start should have comment or be unexported (golint)
    • Line 129: warning: exported function New should have comment or be unexported (golint)
    • velocity/backend/pkg/builder/writer.go
    • Line 13: warning: exported type Emitter should have comment or be unexported (golint)
    • Line 17: warning: exported function NewEmitter should have comment or be unexported (golint)
    • Line 23: warning: exported method Emitter.GetStreamWriter should have comment or be unexported (golint)
    • Line 59: warning: exported method Emitter.GetStepWriter should have comment or be unexported (golint)
    • Line 78: warning: exported method Emitter.GetTaskWriter should have comment or be unexported (golint)
    • Line 115: warning: exported type StateWriter should have comment or be unexported (golint)
    • Line 130: warning: exported type BufferedWriter should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/config/step.go
    • Line 10: warning: exported type Step should have comment or be unexported (golint)
    • Line 12: warning: exported type BaseStep should have comment or be unexported (golint)
    • Line 17: warning: exported type StepSetup should have comment or be unexported (golint)
    • Line 19: warning: exported type StepBlueprint should have comment or be unexported (golint)
    • Line 25: warning: exported type StepDockerRun should have comment or be unexported (golint)
    • Line 35: warning: exported type StepDockerPush should have comment or be unexported (golint)
    • Line 40: warning: exported type StepDockerCompose should have comment or be unexported (golint)
    • Line 45: warning: exported type StepDockerBuild should have comment or be unexported (golint)
    • velocity/backend/pkg/velocity/build/step-docker-compose.go
    • Line 18: warning: exported type StepDockerCompose should have comment or be unexported (golint)
    • Line 26: warning: exported function NewStepDockerCompose should have comment or be unexported (golint)
    • Line 35: warning: exported method StepDockerCompose.GetDetails should have comment or be unexported (golint)
    • Line 45: warning: exported method StepDockerCompose.Validate should have comment or be unexported (golint)
    • Line 49: warning: exported method StepDockerCompose.SetParams should have comment or be unexported (golint)
    • Line 82: warning: exported method StepDockerCompose.Execute should have comment or be unexported (golint)
    • Line 139: warning: exported method StepDockerCompose.Stop should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign98%

IneffAssign detects ineffectual assignments in Go code.


misspell100%

Misspell Finds commonly misspelled English words

No problems detected. Good job!