Sign upLogin

efritz/ij

View on GitHub
Star
registry/ecr.go

Summary

Maintainability
A
1 hr
Test Coverage

Function getAWSCredentials has 6 return statements (exceeds 4 allowed).
Open

func getAWSCredentials(
    env environment.Environment,
    registry *config.ECRRegistry,
) (*awsCredentials, error) {
    accessKeyID, err := env.ExpandString(registry.AccessKeyID)
Severity: Major
Found in registry/ecr.go - About 40 mins to fix

    Function newECRLogin has 5 arguments (exceeds 4 allowed). Consider refactoring.
    Open

        ctx context.Context,
    logger logging.Logger,
    env environment.Environment,
    registry *config.ECRRegistry,
    runner command.Runner,
    Severity: Minor
    Found in registry/ecr.go - About 35 mins to fix

      exported function NewECRLogin should have comment or be unexported
      Open

      func NewECRLogin(
      Severity: Minor
      Found in registry/ecr.go by golint

      exported const ECRServerFormat should have comment (or a comment on this block) or be unexported
      Open

          ECRServerFormat = "https://%s.dkr.ecr.%s.amazonaws.com"
      Severity: Minor
      Found in registry/ecr.go by golint

      There are no issues that match your filters.

      Category
      Status