vshn/appcat-service-postgresql

View on GitHub

Showing 6 of 24 total issues

Function EnsureHelmReleaseFn has 51 lines of code (exceeds 50 allowed). Consider refactoring.
Open

func EnsureHelmReleaseFn(labelSet labels.Set) func(ctx context.Context) error {
    return func(ctx context.Context) error {
        kube := GetClientFromContext(ctx)
        instance := GetInstanceFromContext(ctx)
        config := GetConfigFromContext(ctx)
Severity: Minor
Found in operator/steps/helmrelease.go - About 1 hr to fix

    exported type GenerationStatus should have comment or be unexported
    Open

    type GenerationStatus struct {
    Severity: Minor
    Found in apis/postgresql/v1alpha1/status.go by golint

    Your code does not pass gofmt in 1 place. Go fmt your code!
    Open

    package standalone

    TODO found
    Confirmed

            // TODO: maybe a better semver comparison later on?
    Severity: Minor
    Found in operator/steps/helmrelease.go by fixme

    TODO found
    Confirmed

                            // TODO: add step to provision S3 bucket
    Severity: Minor
    Found in operator/standalone/provisioning.go by fixme

    TODO found
    Confirmed

        // TODO: Add APPUiO cloud organization label that identifies ownership.
    Severity: Minor
    Found in operator/standalone/provisioning.go by fixme
    Severity
    Category
    Status
    Source
    Language