Preparing report...

Report for github.com/philippheuer/envcli

A    Great!    Found 20 issues across 22 files

Tweet

gofmt77%

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


go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!


gocyclo95%

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.

    • envcli/app.go
    • Line 48: warning: cyclomatic complexity 33 of function main() is high (> 15) (gocyclo)

golint13%

Golint is a linter for Go source code.

    • envcli/pkg/config/config.go
    • Line 23: warning: comment on exported function LoadProjectConfig should be of the form "LoadProjectConfig ..." (golint)
    • Line 40: warning: comment on exported function LoadPropertyConfig should be of the form "LoadPropertyConfig ..." (golint)
    • Line 47: warning: comment on exported function LoadPropertyConfigFile should be of the form "LoadPropertyConfigFile ..." (golint)
    • Line 65: warning: comment on exported function SavePropertyConfig should be of the form "SavePropertyConfig ..." (golint)
    • Line 72: warning: comment on exported function SavePropertyConfigFile should be of the form "SavePropertyConfigFile ..." (golint)
    • Line 86: warning: comment on exported function SetPropertyConfigEntry should be of the form "SetPropertyConfigEntry ..." (golint)
    • Line 103: warning: comment on exported function GetPropertyConfigEntry should be of the form "GetPropertyConfigEntry ..." (golint)
    • Line 119: warning: comment on exported function UnsetPropertyConfigEntry should be of the form "UnsetPropertyConfigEntry ..." (golint)
    • Line 136: warning: comment on exported function GetProjectOrWorkingDirectory should be of the form "GetProjectOrWorkingDirectory ..." (golint)
    • Line 147: warning: comment on exported function GetProjectDirectory should be of the form "GetProjectDirectory ..." (golint)
    • Line 177: warning: comment on exported function MergeConfigurations should be of the form "MergeConfigurations ..." (golint)
    • Line 195: warning: comment on exported function GetCommandConfiguration should be of the form "GetCommandConfiguration ..." (golint)
    • envcli/pkg/sentry/sentry.go
    • Line 7: warning: comment on exported function InitializeSentryIO should be of the form "InitializeSentryIO ..." (golint)
    • Line 17: warning: exported function HandleError should have comment or be unexported (golint)
    • envcli/pkg/container_runtime/util.go
    • Line 1: warning: don't use an underscore in package name (golint)
    • Line 14: warning: comment on exported function IsCIEnvironment should be of the form "IsCIEnvironment ..." (golint)
    • Line 33: warning: comment on exported function IsPodman should be of the form "IsPodman ..." (golint)
    • Line 46: warning: comment on exported function IsDockerNative should be of the form "IsDockerNative ..." (golint)
    • Line 149: warning: comment on exported function InArray should be of the form "InArray ..." (golint)
    • envcli/pkg/updater/model.go
    • Line 3: warning: comment on exported type ApplicationUpdater should be of the form "ApplicationUpdater ..." (with optional leading article) (golint)
    • envcli/pkg/config/model.go
    • Line 3: warning: comment on exported type ConfigurationLoader should be of the form "ConfigurationLoader ..." (with optional leading article) (golint)
    • Line 9: warning: comment on exported type ProjectConfigrationFile should be of the form "ProjectConfigrationFile ..." (with optional leading article) (golint)
    • Line 17: warning: comment on exported type RunConfigurationEntry should be of the form "RunConfigurationEntry ..." (with optional leading article) (golint)
    • Line 58: warning: exported type CachingEntry should have comment or be unexported (golint)
    • Line 71: warning: comment on exported type PropertyConfigurationFile should be of the form "PropertyConfigurationFile ..." (with optional leading article) (golint)
    • envcli/pkg/config/utils.go
    • Line 11: warning: comment on exported function GetWorkingDirectory should be of the form "GetWorkingDirectory ..." (golint)
    • Line 26: warning: comment on exported function InArray should be of the form "InArray ..." (golint)
    • Line 49: warning: comment on exported function GetOrDefault should be of the form "GetOrDefault ..." (golint)
    • envcli/pkg/util/path.go
    • Line 11: warning: comment on exported function CreateDirectory should be of the form "CreateDirectory ..." (golint)
    • Line 18: warning: comment on exported function GetPathRelativeToDirectory should be of the form "GetPathRelativeToDirectory ..." (golint)
    • Line 29: warning: comment on exported function GetExecutionDirectory should be of the form "GetExecutionDirectory ..." (golint)
    • Line 44: warning: comment on exported function GetWorkingDirectory should be of the form "GetWorkingDirectory ..." (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign95%

IneffAssign detects ineffectual assignments in Go code.


misspell86%

Misspell Finds commonly misspelled English words