Preparing report...

Report for github.com/davidji99/terraform-provider-herokux

A+    Excellent!    Found 26 issues across 161 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!


golint86%

Golint is a linter for Go source code.

    • terraform-provider-herokux/api/connect/connection.go
    • Line 77: warning: exported type ConnectionOData should have comment or be unexported (golint)
    • Line 82: warning: exported type ConnectionDatabase should have comment or be unexported (golint)
    • Line 88: warning: exported type ConnectionSalesforceInfo should have comment or be unexported (golint)
    • Line 96: warning: exported type ConnectionFeatures should have comment or be unexported (golint)
    • Line 116: warning: exported type ConnectionUpdateRequest should have comment or be unexported (golint)
    • terraform-provider-herokux/api/platform/types.go
    • Line 3: warning: exported type WebhookLevel should have comment or be unexported (golint)
    • Line 5: warning: exported var WebhookLevels should have comment or be unexported (golint)
    • Line 13: warning: exported method WebhookLevel.ToString should have comment or be unexported (golint)
    • Line 17: warning: exported type WebhookEventType should have comment or be unexported (golint)
    • Line 19: warning: exported var WebhookEventTypes should have comment or be unexported (golint)
    • Line 45: warning: exported method WebhookEventType.ToString should have comment or be unexported (golint)
    • terraform-provider-herokux/api/kafka/helper.go
    • Line 18: warning: comment on exported const RetentionTimeDuragionRegexStricterWithDisable should be of the form "RetentionTimeDuragionRegexStricterWithDisable ..." (golint)
    • Line 22: warning: exported const RetentionTimeDisableVal should have comment (or a comment on this block) or be unexported (golint)
    • Line 24: warning: comment on exported const WeekMultiplier should be of the form "WeekMultiplier ..." (golint)
    • terraform-provider-herokux/api/kolkrabbi/account.go
    • Line 8: warning: exported type AccountInfo should have comment or be unexported (golint)
    • Line 16: warning: exported type AccountInfoGithub should have comment or be unexported (golint)
    • Line 21: warning: exported type AccountInfoHeroku should have comment or be unexported (golint)
    • Line 25: warning: exported method Kolkrabbi.GetAccountInfo should have comment or be unexported (golint)
    • terraform-provider-herokux/api/kolkrabbi/pipeline.go
    • Line 26: warning: exported type PipelineGHIntegrationUser should have comment or be unexported (golint)
    • Line 32: warning: exported type PipelineGHIntegrationHerokuUser should have comment or be unexported (golint)
    • Line 36: warning: exported type PipelineGHIntegrationGithubUser should have comment or be unexported (golint)
    • Line 40: warning: exported type PipelineRepository should have comment or be unexported (golint)
    • terraform-provider-herokux/helper/test/config.go
    • Line 11: warning: exported type TestConfigKey should have comment or be unexported (golint)
    • Line 14: warning: exported const TestConfigHerokuxAPIKey should have comment (or a comment on this block) or be unexported (golint)
    • Line 60: warning: exported type TestConfig should have comment or be unexported (golint)
    • Line 62: warning: exported function NewTestConfig should have comment or be unexported (golint)
    • Line 66: warning: exported method TestConfig.Get should have comment or be unexported (golint)
    • Line 76: warning: exported method TestConfig.GetOrSkip should have comment or be unexported (golint)
    • Line 85: warning: exported method TestConfig.GetOrAbort should have comment or be unexported (golint)
    • Line 94: warning: exported method TestConfig.SkipUnlessAccTest should have comment or be unexported (golint)
    • Line 101: warning: exported method TestConfig.GetAddonIDorSkip should have comment or be unexported (golint)
    • Line 105: warning: exported method TestConfig.GetAppIDorSkip should have comment or be unexported (golint)
    • Line 109: warning: exported method TestConfig.GetDBNameorSkip should have comment or be unexported (golint)
    • Line 113: warning: exported method TestConfig.GetKafkaIDorSkip should have comment or be unexported (golint)
    • Line 117: warning: exported method TestConfig.GetCustomAPIKeyorSkip should have comment or be unexported (golint)
    • Line 121: warning: exported method TestConfig.GetTeamIDorSkip should have comment or be unexported (golint)
    • Line 125: warning: exported method TestConfig.GetRedisIDorSkip should have comment or be unexported (golint)
    • Line 129: warning: exported method TestConfig.GetPostgresIDorSkip should have comment or be unexported (golint)
    • Line 133: warning: exported method TestConfig.GetConnectIDorSkip should have comment or be unexported (golint)
    • Line 137: warning: exported method TestConfig.GetImageIDorSkip should have comment or be unexported (golint)
    • Line 141: warning: exported method TestConfig.GetPipelineIDorSkip should have comment or be unexported (golint)
    • Line 145: warning: exported method TestConfig.GetGithubOrgRepoorSkip should have comment or be unexported (golint)
    • Line 149: warning: exported method TestConfig.GetUserEmailorSkip should have comment or be unexported (golint)
    • Line 153: warning: exported method TestConfig.GetAnyOrganizationOrSkip should have comment or be unexported (golint)
    • Line 157: warning: exported method TestConfig.GetRunE2ETestsOrSkip should have comment or be unexported (golint)
    • terraform-provider-herokux/api/redis/config.go
    • Line 6: warning: exported const DisableNotifyKeyspaceEvents should have comment (or a comment on this block) or be unexported (golint)
    • Line 31: warning: comment on exported type ConfigTimeout should be of the form "ConfigTimeout ..." (with optional leading article) (golint)
    • Line 38: warning: exported type ConfigStandbySegvWorkaround should have comment or be unexported (golint)

gocyclo98%

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.


ineffassign100%

IneffAssign detects ineffectual assignments in Go code.

No problems detected. Good job!


license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


misspell99%

Misspell Finds commonly misspelled English words