Preparing report...

Report for github.com/secrethub/secrethub-go

A+    Excellent!    Found 38 issues across 166 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!


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.


golint77%

Golint is a linter for Go source code.

    • secrethub-go/pkg/secrethub/fakeclient/idp_link.go
    • Line 10: warning: exported type IDPLinkService should have comment or be unexported (golint)
    • Line 14: warning: exported method IDPLinkService.GCP should have comment or be unexported (golint)
    • Line 18: warning: exported type IDPLinkGCPService should have comment or be unexported (golint)
    • Line 27: warning: exported method IDPLinkGCPService.Create should have comment or be unexported (golint)
    • Line 31: warning: exported method IDPLinkGCPService.List should have comment or be unexported (golint)
    • Line 35: warning: exported method IDPLinkGCPService.Get should have comment or be unexported (golint)
    • Line 39: warning: exported method IDPLinkGCPService.Exists should have comment or be unexported (golint)
    • Line 43: warning: exported method IDPLinkGCPService.Delete should have comment or be unexported (golint)
    • Line 47: warning: exported method IDPLinkGCPService.AuthorizationCodeListener should have comment or be unexported (golint)
    • Line 51: warning: exported type IDPLinkIterator should have comment or be unexported (golint)
    • Line 57: warning: exported method IDPLinkIterator.Next should have comment or be unexported (golint)
    • secrethub-go/internals/api/idp_link.go
    • Line 11: warning: exported var ErrInvalidIDPLinkType should have comment or be unexported (golint)
    • Line 20: warning: exported type CreateIdentityProviderLinkGCPRequest should have comment or be unexported (golint)
    • Line 25: warning: exported type IdentityProviderLinkType should have comment or be unexported (golint)
    • Line 28: warning: exported const IdentityProviderLinkGCP should have comment (or a comment on this block) or be unexported (golint)
    • Line 46: warning: exported type OAuthConfig should have comment or be unexported (golint)
    • secrethub-go/pkg/secrethub/fakeclient/me.go
    • Line 8: warning: exported type MeService should have comment or be unexported (golint)
    • Line 15: warning: exported method MeService.GetUser should have comment or be unexported (golint)
    • Line 19: warning: exported method MeService.SendVerificationEmail should have comment or be unexported (golint)
    • Line 23: warning: exported method MeService.ListRepos should have comment or be unexported (golint)
    • Line 27: warning: exported method MeService.RepoIterator should have comment or be unexported (golint)
    • secrethub-go/internals/api/credential.go
    • Line 50: warning: exported const ShortCredentialFingerprintMinimumLength should have comment (or a comment on this block) or be unexported (golint)
    • Line 262: warning: comment on exported type CredentialProofGCPServiceAccount should be of the form "CredentialProofGCPServiceAccount ..." (with optional leading article) (golint)
    • secrethub-go/pkg/secrethub/iterator/iterator.go
    • Line 16: warning: exported type Paginator should have comment or be unexported (golint)
    • Line 20: warning: exported type PaginatorConstructor should have comment or be unexported (golint)
    • Line 22: warning: exported type Iterator should have comment or be unexported (golint)
    • Line 30: warning: exported function New should have comment or be unexported (golint)
    • Line 39: warning: exported method Iterator.Next should have comment or be unexported (golint)
    • secrethub-go/internals/gcp/service_creator.go
    • Line 53: warning: exported method CredentialCreator.Wrap should have comment or be unexported (golint)
    • Line 61: warning: exported method CredentialCreator.Export should have comment or be unexported (golint)
    • Line 66: warning: exported method CredentialCreator.Type should have comment or be unexported (golint)
    • Line 70: warning: exported method CredentialCreator.AddProof should have comment or be unexported (golint)
    • secrethub-go/pkg/secrethub/fakeclient/credentials.go
    • Line 10: warning: exported type CredentialService should have comment or be unexported (golint)
    • Line 16: warning: exported method CredentialService.Create should have comment or be unexported (golint)
    • Line 20: warning: exported method CredentialService.Disable should have comment or be unexported (golint)
    • Line 24: warning: exported method CredentialService.List should have comment or be unexported (golint)
    • Line 28: warning: exported type CredentialIterator should have comment or be unexported (golint)
    • Line 34: warning: exported method CredentialIterator.Next should have comment or be unexported (golint)
    • secrethub-go/pkg/secrethub/fakeclient/org_member.go
    • Line 20: warning: exported method OrgMemberService.Invite should have comment or be unexported (golint)
    • Line 24: warning: exported method OrgMemberService.Get should have comment or be unexported (golint)
    • Line 28: warning: exported method OrgMemberService.Update should have comment or be unexported (golint)
    • Line 32: warning: exported method OrgMemberService.Revoke should have comment or be unexported (golint)
    • Line 36: warning: exported method OrgMemberService.List should have comment or be unexported (golint)
    • Line 40: warning: exported method OrgMemberService.Iterator should have comment or be unexported (golint)
    • secrethub-go/internals/oauthorizer/callback_handler.go
    • Line 14: warning: exported type CallbackHandler should have comment or be unexported (golint)
    • Line 24: warning: exported function NewCallbackHandler should have comment or be unexported (golint)
    • Line 44: warning: exported method CallbackHandler.ListenURL should have comment or be unexported (golint)
    • Line 48: warning: exported method CallbackHandler.AuthorizeURL 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!