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!

gocyclo91%

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.

• optimize-controller/api/apps/v1alpha1/application_defaults.go
  • Line 87: warning: cyclomatic complexity 17 of function (*Goal).Default() is high (> 15) (gocyclo)

• optimize-controller/redskyctl/internal/commands/check/controller.go
  • Line 61: warning: cyclomatic complexity 17 of function (*ControllerOptions).CheckController() is high (> 15) (gocyclo)

• optimize-controller/api/v1alpha1/zz_generated.deepcopy.go
  • Line 718: warning: cyclomatic complexity 22 of function (*TrialSpec).DeepCopyInto() is high (> 15) (gocyclo)

• optimize-controller/redskyctl/internal/commands/run/model.go
  • Line 266: warning: cyclomatic complexity 22 of function (previewModel).Update() is high (> 15) (gocyclo)

• optimize-controller/controllers/trial_job_controller.go
  • Line 177: warning: cyclomatic complexity 19 of function (*TrialJobReconciler).applyJobStatus() is high (> 15) (gocyclo)

• optimize-controller/internal/experiment/generation/transformer.go
  • Line 74: warning: cyclomatic complexity 17 of function (*Transformer).Transform() is high (> 15) (gocyclo)

• optimize-controller/internal/setup/job.go
  • Line 51: warning: cyclomatic complexity 26 of function NewJob() is high (> 15) (gocyclo)

• optimize-controller/api/v1beta1/zz_generated.deepcopy.go
  • Line 737: warning: cyclomatic complexity 18 of function (*TrialSpec).DeepCopyInto() is high (> 15) (gocyclo)

• optimize-controller/redskyctl/internal/commands/check/experiment.go
  • Line 136: warning: cyclomatic complexity 45 of function (*linter).Visit() is high (> 15) (gocyclo)

• optimize-controller/api/v1alpha1/zz_generated.conversion.go
  • Line 37: warning: cyclomatic complexity 65 of function RegisterConversions() is high (> 15) (gocyclo)
  • Line 563: warning: cyclomatic complexity 18 of function autoConvert_v1alpha1_ExperimentSpec_To_v1beta1_ExperimentSpec() is high (> 15) (gocyclo)
  • Line 635: warning: cyclomatic complexity 18 of function autoConvert_v1beta1_ExperimentSpec_To_v1alpha1_ExperimentSpec() is high (> 15) (gocyclo)

• optimize-controller/redskyctl/internal/commands/run/run.go
  • Line 124: warning: cyclomatic complexity 29 of function (*Options).Update() is high (> 15) (gocyclo)

• optimize-controller/redskyctl/internal/commands/run/form/form.go
  • Line 62: warning: cyclomatic complexity 21 of function (Fields).Update() is high (> 15) (gocyclo)

• optimize-controller/internal/trial/status.go
  • Line 82: warning: cyclomatic complexity 31 of function summarize() is high (> 15) (gocyclo)

• optimize-controller/internal/server/server.go
  • Line 45: warning: cyclomatic complexity 21 of function FromCluster() is high (> 15) (gocyclo)

• optimize-controller/internal/experiment/visitor.go
  • Line 36: warning: cyclomatic complexity 20 of function Walk() is high (> 15) (gocyclo)

• optimize-controller/controllers/server_controller.go
  • Line 86: warning: cyclomatic complexity 18 of function (*ServerReconciler).Reconcile() is high (> 15) (gocyclo)

• optimize-controller/redskyctl/internal/commands/experiments/experiments.go
  • Line 322: warning: cyclomatic complexity 30 of function (*experimentsMeta).ExtractValue() is high (> 15) (gocyclo)
  • Line 129: warning: cyclomatic complexity 20 of function parseNames() is high (> 15) (gocyclo)

• optimize-controller/redskyctl/internal/commands/export/export.go
  • Line 270: warning: cyclomatic complexity 16 of function (*Options).runner() is high (> 15) (gocyclo)

golint71%

Golint is a linter for Go source code.

• optimize-controller/redskyctl/internal/commands/completion/completion.go
  • Line 30: warning: exported function NewCommand should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/choiceinput/choiceinput.go
  • Line 28: warning: exported type Model should have comment or be unexported (golint)
  • Line 46: warning: exported function NewModel should have comment or be unexported (golint)
  • Line 59: warning: exported method Model.Update should have comment or be unexported (golint)
  • Line 94: warning: exported method Model.IsSelected should have comment or be unexported (golint)
  • Line 98: warning: exported method Model.Select should have comment or be unexported (golint)
  • Line 116: warning: exported method Model.SelectOnly should have comment or be unexported (golint)
  • Line 122: warning: exported method Model.View should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/ping/ping.go
  • Line 35: warning: exported type Options should have comment or be unexported (golint)
  • Line 44: warning: comment on exported function NewCommand should be of the form "NewCommand ..." (golint)

• optimize-controller/redskyctl/internal/commands/run/form/multichoicefield.go
  • Line 24: warning: exported type MultiChoiceFieldValidator should have comment or be unexported (golint)
  • Line 28: warning: exported type MultiChoiceField should have comment or be unexported (golint)
  • Line 36: warning: exported function NewMultiChoiceField should have comment or be unexported (golint)
  • Line 51: warning: exported method MultiChoiceField.Update should have comment or be unexported (golint)
  • Line 68: warning: exported method MultiChoiceField.View should have comment or be unexported (golint)
  • Line 78: warning: exported method MultiChoiceField.Validate should have comment or be unexported (golint)

• optimize-controller/internal/application/generator.go
  • Line 52: warning: exported method Generator.Execute should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/form/textfield.go
  • Line 24: warning: exported type TextFieldValidator should have comment or be unexported (golint)
  • Line 28: warning: exported type TextField should have comment or be unexported (golint)
  • Line 37: warning: exported function NewTextField should have comment or be unexported (golint)
  • Line 52: warning: exported method TextField.Update should have comment or be unexported (golint)
  • Line 76: warning: exported method TextField.View should have comment or be unexported (golint)
  • Line 86: warning: exported method TextField.Validate should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/pager/pager.go
  • Line 26: warning: exported type ExitMsg should have comment or be unexported (golint)
  • Line 28: warning: exported type Model should have comment or be unexported (golint)
  • Line 39: warning: exported function NewModel should have comment or be unexported (golint)
  • Line 43: warning: exported method Model.Update should have comment or be unexported (golint)
  • Line 79: warning: exported method Model.View should have comment or be unexported (golint)
  • Line 89: warning: exported method Model.Focus should have comment or be unexported (golint)
  • Line 93: warning: exported method Model.Focused should have comment or be unexported (golint)
  • Line 97: warning: exported method Model.Blur should have comment or be unexported (golint)

• optimize-controller/internal/experiment/generation/prometheus.go
  • Line 29: warning: exported type PrometheusMetricsSource should have comment or be unexported (golint)
  • Line 35: warning: exported method PrometheusMetricsSource.Metrics should have comment or be unexported (golint)
  • Line 49: warning: exported type BuiltInPrometheus should have comment or be unexported (golint)
  • Line 61: warning: exported method BuiltInPrometheus.Update should have comment or be unexported (golint)

• optimize-controller/internal/experiment/generation/requests.go
  • Line 32: warning: exported type RequestsMetricsSource should have comment or be unexported (golint)
  • Line 38: warning: exported method RequestsMetricsSource.Metrics should have comment or be unexported (golint)

• optimize-controller/api/v1alpha1/experiment_conversion.go
  • Line 40: warning: exported method Experiment.ConvertTo should have comment or be unexported (golint)
  • Line 48: warning: exported method Experiment.ConvertFrom should have comment or be unexported (golint)
  • Line 56: warning: exported function Convert_v1alpha1_ExperimentSpec_To_v1beta1_ExperimentSpec should have comment or be unexported (golint)
  • Line 56: warning: don't use underscores in Go names; func Convert_v1alpha1_ExperimentSpec_To_v1beta1_ExperimentSpec should be ConvertV1alpha1ExperimentSpecToV1beta1ExperimentSpec (golint)
  • Line 66: warning: exported function Convert_v1beta1_ExperimentSpec_To_v1alpha1_ExperimentSpec should have comment or be unexported (golint)
  • Line 66: warning: don't use underscores in Go names; func Convert_v1beta1_ExperimentSpec_To_v1alpha1_ExperimentSpec should be ConvertV1beta1ExperimentSpecToV1alpha1ExperimentSpec (golint)
  • Line 76: warning: exported function Convert_v1alpha1_Parameter_To_v1beta1_Parameter should have comment or be unexported (golint)
  • Line 76: warning: don't use underscores in Go names; func Convert_v1alpha1_Parameter_To_v1beta1_Parameter should be ConvertV1alpha1ParameterToV1beta1Parameter (golint)
  • Line 91: warning: exported function Convert_v1beta1_Parameter_To_v1alpha1_Parameter should have comment or be unexported (golint)
  • Line 91: warning: don't use underscores in Go names; func Convert_v1beta1_Parameter_To_v1alpha1_Parameter should be ConvertV1beta1ParameterToV1alpha1Parameter (golint)
  • Line 105: warning: exported function Convert_v1beta1_Metric_To_v1alpha1_Metric should have comment or be unexported (golint)
  • Line 105: warning: don't use underscores in Go names; func Convert_v1beta1_Metric_To_v1alpha1_Metric should be ConvertV1beta1MetricToV1alpha1Metric (golint)
  • Line 134: warning: exported function Convert_v1alpha1_Metric_To_v1beta1_Metric should have comment or be unexported (golint)
  • Line 134: warning: don't use underscores in Go names; func Convert_v1alpha1_Metric_To_v1beta1_Metric should be ConvertV1alpha1MetricToV1beta1Metric (golint)

• optimize-controller/redskyctl/internal/commands/generate/rbac.go
  • Line 44: warning: exported type RBACOptions should have comment or be unexported (golint)
  • Line 61: warning: exported function NewRBACCommand should have comment or be unexported (golint)
  • Line 93: warning: exported method RBACOptions.Complete should have comment or be unexported (golint)

• optimize-controller/controllers/setup_controller.go
  • Line 50: warning: exported method SetupReconciler.Reconcile should have comment or be unexported (golint)
  • Line 82: warning: exported method SetupReconciler.SetupWithManager should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/initialize/initialize.go
  • Line 53: warning: exported method Options.Initialize should have comment or be unexported (golint)

• optimize-controller/api/apps/v1alpha1/application_defaults.go
  • Line 35: warning: exported const StormForgerAccessTokenSecretName should have comment (or a comment on this block) or be unexported (golint)
  • Line 39: warning: comment on exported function RegisterDefaults should be of the form "RegisterDefaults ..." (golint)
  • Line 45: warning: exported method Application.Default should have comment or be unexported (golint)
  • Line 55: warning: exported method Scenario.Default should have comment or be unexported (golint)
  • Line 70: warning: exported method Objective.Default should have comment or be unexported (golint)
  • Line 87: warning: exported method Goal.Default should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/run.go
  • Line 46: warning: exported type Options should have comment or be unexported (golint)
  • Line 100: warning: exported method Options.Init should have comment or be unexported (golint)
  • Line 124: warning: exported method Options.Update should have comment or be unexported (golint)

• optimize-controller/api/v1alpha1/doc.go
  • Line 17: warning: package comment should be of the form "Package v1alpha1 ..." (golint)

• optimize-controller/redskyctl/internal/commands/fix/fix.go
  • Line 32: warning: exported type Options should have comment or be unexported (golint)
  • Line 38: warning: exported function NewCommand should have comment or be unexported (golint)
  • Line 56: warning: exported method Options.Fix should have comment or be unexported (golint)

• optimize-controller/internal/experiment/generation/replicas.go
  • Line 38: warning: exported method ReplicaSelector.Default should have comment or be unexported (golint)
  • Line 48: warning: exported method ReplicaSelector.Map should have comment or be unexported (golint)

• optimize-controller/internal/experiment/experiment.go
  • Line 123: warning: exported function IsFinished should have comment or be unexported (golint)
  • Line 134: warning: exported function ApplyCondition should have comment or be unexported (golint)

• optimize-controller/controllers/experiment_controller.go
  • Line 46: warning: exported method ExperimentReconciler.Reconcile should have comment or be unexported (golint)
  • Line 74: warning: exported method ExperimentReconciler.SetupWithManager should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/form/completion.go
  • Line 31: warning: exported type Completions should have comment or be unexported (golint)
  • Line 126: warning: exported method CompletionModel.View should have comment or be unexported (golint)
  • Line 199: warning: exported type FileCompletions should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/view.go
  • Line 36: warning: exported const ScenarioTypeStormForger should have comment (or a comment on this block) or be unexported (golint)

• optimize-controller/internal/experiment/generation/transformer.go
  • Line 140: warning: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (golint)

• optimize-controller/internal/sfio/runtime.go
  • Line 51: warning: comment on exported type ObjectSlice should be of the form "ObjectSlice ..." (with optional leading article) (golint)

• optimize-controller/config/config_files.go
  • Line 21: warning: comment on exported var Content should be of the form "Content ..." (golint)

• optimize-controller/api/v1alpha1/trial_conversion.go
  • Line 27: warning: exported method Trial.ConvertTo should have comment or be unexported (golint)
  • Line 35: warning: exported method Trial.ConvertFrom should have comment or be unexported (golint)
  • Line 43: warning: exported function Convert_v1alpha1_Trial_To_v1beta1_Trial should have comment or be unexported (golint)
  • Line 43: warning: don't use underscores in Go names; func Convert_v1alpha1_Trial_To_v1beta1_Trial should be ConvertV1alpha1TrialToV1beta1Trial (golint)
  • Line 74: warning: exported function Convert_v1beta1_Trial_To_v1alpha1_Trial should have comment or be unexported (golint)
  • Line 74: warning: don't use underscores in Go names; func Convert_v1beta1_Trial_To_v1alpha1_Trial should be ConvertV1beta1TrialToV1alpha1Trial (golint)
  • Line 105: warning: exported function Convert_v1alpha1_TrialSpec_To_v1beta1_TrialSpec should have comment or be unexported (golint)
  • Line 105: warning: don't use underscores in Go names; func Convert_v1alpha1_TrialSpec_To_v1beta1_TrialSpec should be ConvertV1alpha1TrialSpecToV1beta1TrialSpec (golint)
  • Line 113: warning: exported function Convert_v1beta1_TrialSpec_To_v1alpha1_TrialSpec should have comment or be unexported (golint)
  • Line 113: warning: don't use underscores in Go names; func Convert_v1beta1_TrialSpec_To_v1alpha1_TrialSpec should be ConvertV1beta1TrialSpecToV1alpha1TrialSpec (golint)
  • Line 121: warning: exported function Convert_v1beta1_TrialStatus_To_v1alpha1_TrialStatus should have comment or be unexported (golint)
  • Line 121: warning: don't use underscores in Go names; func Convert_v1beta1_TrialStatus_To_v1alpha1_TrialStatus should be ConvertV1beta1TrialStatusToV1alpha1TrialStatus (golint)

• optimize-controller/internal/controller/negotiation.go
  • Line 64: warning: exported method ConversionSerializer.DecoderToVersion should have comment or be unexported (golint)
  • Line 74: warning: exported method ConversionDecoder.Decode should have comment or be unexported (golint)

• optimize-controller/internal/experiment/generation/custom.go
  • Line 32: warning: exported type CustomSource should have comment or be unexported (golint)
  • Line 41: warning: exported method CustomSource.Update should have comment or be unexported (golint)
  • Line 84: warning: exported method CustomSource.Metrics should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/reset/reset.go
  • Line 37: warning: exported function NewCommand should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/check/version.go
  • Line 35: warning: exported var GitHubReleasesURL should have comment or be unexported (golint)
  • Line 113: warning: exported type ReleaseList should have comment or be unexported (golint)
  • Line 115: warning: exported type Release should have comment or be unexported (golint)
  • Line 123: warning: exported type Asset should have comment or be unexported (golint)
  • Line 128: warning: exported type APIError should have comment or be unexported (golint)
  • Line 134: warning: exported method ReleaseList.Latest should have comment or be unexported (golint)
  • Line 144: warning: exported method Release.AssetByName should have comment or be unexported (golint)

• optimize-controller/controllers/metric_controller.go
  • Line 55: warning: exported method MetricReconciler.Reconcile should have comment or be unexported (golint)
  • Line 75: warning: exported method MetricReconciler.SetupWithManager should have comment or be unexported (golint)

• optimize-controller/controllers/server_controller.go
  • Line 86: warning: exported method ServerReconciler.Reconcile should have comment or be unexported (golint)
  • Line 160: warning: exported method ServerReconciler.SetupWithManager should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/debug/metric.go
  • Line 74: warning: exported method MetricQueryOptions.Debug should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/authorize_cluster/generator.go
  • Line 17: warning: don't use an underscore in package name (golint)

• optimize-controller/redskyctl/internal/commands/generate/experiment.go
  • Line 35: warning: exported type ExperimentOptions should have comment or be unexported (golint)
  • Line 50: warning: exported function NewExperimentCommand should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/kustomize/consts/redskyconfig.go
  • Line 21: warning: exported function GetRedSkyFieldSpecs should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/grant_permissions/generator.go
  • Line 17: warning: don't use an underscore in package name (golint)

• optimize-controller/redskyctl/internal/kustomize/options.go
  • Line 35: warning: exported type Option should have comment or be unexported (golint)
  • Line 42: warning: comment on exported var BuildImage should be of the form "BuildImage ..." (golint)
  • Line 233: warning: exported function WithImagePullPolicy should have comment or be unexported (golint)
  • Line 258: warning: exported function WithFS should have comment or be unexported (golint)
  • Line 267: warning: exported function WithResourceNames should have comment or be unexported (golint)

• optimize-controller/api/apps/v1alpha1/application_types.go
  • Line 203: warning: exported const LatencyMinimum should have comment (or a comment on this block) or be unexported (golint)
  • Line 226: warning: exported const ErrorRateRequests should have comment (or a comment on this block) or be unexported (golint)
  • Line 244: warning: exported const DurationTrial should have comment (or a comment on this block) or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/form/exitfield.go
  • Line 23: warning: exported type ExitField should have comment or be unexported (golint)
  • Line 31: warning: exported function NewExitField should have comment or be unexported (golint)
  • Line 35: warning: exported method ExitField.Update should have comment or be unexported (golint)
  • Line 43: warning: exported method ExitField.Focus should have comment or be unexported (golint)
  • Line 47: warning: exported method ExitField.Focused should have comment or be unexported (golint)
  • Line 51: warning: exported method ExitField.Blur should have comment or be unexported (golint)
  • Line 55: warning: exported method ExitField.View should have comment or be unexported (golint)
  • Line 59: warning: exported method ExitField.Validate should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/form/validation.go
  • Line 39: warning: exported type Required should have comment or be unexported (golint)
  • Line 46: warning: exported method Required.ValidateTextField should have comment or be unexported (golint)
  • Line 58: warning: exported method Required.ValidateChoiceField should have comment or be unexported (golint)
  • Line 70: warning: exported method Required.ValidateMultiChoiceField should have comment or be unexported (golint)
  • Line 78: warning: exported type URL should have comment or be unexported (golint)
  • Line 84: warning: exported method URL.ValidateTextField should have comment or be unexported (golint)
  • Line 98: warning: exported type File should have comment or be unexported (golint)
  • Line 105: warning: exported method File.ValidateTextField should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/version/version.go
  • Line 125: warning: comment on exported method Options.ControllerVersion should be of the form "ControllerVersion ..." (golint)

• optimize-controller/redskyctl/internal/commands/grant_permissions/grant_permissions.go
  • Line 17: warning: don't use an underscore in package name (golint)

• optimize-controller/internal/experiment/generation/datadog.go
  • Line 26: warning: exported type DatadogMetricsSource should have comment or be unexported (golint)
  • Line 32: warning: exported method DatadogMetricsSource.Metrics should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/out/out.go
  • Line 29: warning: exported type Style should have comment or be unexported (golint)
  • Line 32: warning: exported const Happy should have comment (or a comment on this block) or be unexported (golint)

• optimize-controller/controllers/trial_job_controller.go
  • Line 50: warning: exported method TrialJobReconciler.Reconcile should have comment or be unexported (golint)
  • Line 95: warning: exported method TrialJobReconciler.SetupWithManager should have comment or be unexported (golint)

• optimize-controller/internal/experiment/generation/locust.go
  • Line 30: warning: exported type LocustSource should have comment or be unexported (golint)
  • Line 40: warning: exported method LocustSource.Update should have comment or be unexported (golint)
  • Line 107: warning: exported method LocustSource.Metrics should have comment or be unexported (golint)

• optimize-controller/internal/experiment/generation/stormforger.go
  • Line 34: warning: exported type StormForgerSource should have comment or be unexported (golint)
  • Line 44: warning: exported method StormForgerSource.Update should have comment or be unexported (golint)
  • Line 167: warning: exported method StormForgerSource.Metrics should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/multichoiceinput/multichoiceinput.go
  • Line 28: warning: exported type Model should have comment or be unexported (golint)
  • Line 47: warning: exported function NewModel should have comment or be unexported (golint)
  • Line 59: warning: exported method Model.Update should have comment or be unexported (golint)
  • Line 95: warning: exported method Model.IsSelected should have comment or be unexported (golint)
  • Line 104: warning: exported method Model.Select should have comment or be unexported (golint)
  • Line 114: warning: exported method Model.Unselect should have comment or be unexported (golint)
  • Line 125: warning: exported method Model.SelectOnly should have comment or be unexported (golint)
  • Line 131: warning: exported method Model.Toggle should have comment or be unexported (golint)
  • Line 139: warning: exported method Model.IsHighlighted should have comment or be unexported (golint)
  • Line 143: warning: exported method Model.Highlight should have comment or be unexported (golint)
  • Line 154: warning: exported method Model.Values should have comment or be unexported (golint)
  • Line 162: warning: exported method Model.View should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/run/form/choicefield.go
  • Line 24: warning: exported type ChoiceFieldValidator should have comment or be unexported (golint)
  • Line 28: warning: exported type ChoiceField should have comment or be unexported (golint)
  • Line 36: warning: exported function NewChoiceField should have comment or be unexported (golint)
  • Line 51: warning: exported method ChoiceField.Update should have comment or be unexported (golint)
  • Line 68: warning: exported method ChoiceField.View should have comment or be unexported (golint)
  • Line 78: warning: exported method ChoiceField.Validate should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commander/printer.go
  • Line 91: warning: exported method ResourcePrinterFunc.PrintObj should have comment or be unexported (golint)
  • Line 95: warning: exported method ResourcePrinterFunc.NewPrinter should have comment or be unexported (golint)

• optimize-controller/internal/experiment/generation/duration.go
  • Line 24: warning: exported type DurationMetricsSource should have comment or be unexported (golint)
  • Line 30: warning: exported method DurationMetricsSource.Metrics should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/authorize_cluster/authorize_cluster.go
  • Line 17: warning: don't use an underscore in package name (golint)

• optimize-controller/redskyctl/internal/commands/check/controller.go
  • Line 61: warning: exported method ControllerOptions.CheckController should have comment or be unexported (golint)

• optimize-controller/internal/patch/patch.go
  • Line 75: warning: comment on exported function CreatePatchOperation should be of the form "CreatePatchOperation ..." (golint)

• optimize-controller/redskyctl/internal/commands/generate/application.go
  • Line 31: warning: exported type ApplicationOptions should have comment or be unexported (golint)
  • Line 42: warning: exported function NewApplicationCommand should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/generate/trial.go
  • Line 36: warning: exported type TrialOptions should have comment or be unexported (golint)
  • Line 44: warning: exported function NewTrialCommand should have comment or be unexported (golint)

• optimize-controller/redskyctl/internal/commands/experiments/suggest.go
  • Line 35: warning: exported const DefaultNone should have comment (or a comment on this block) or be unexported (golint)
  • Line 143: warning: exported method SuggestOptions.AddLabels 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!

misspell99%

Misspell Finds commonly misspelled English words

• optimize-controller/redskyctl/internal/commands/generate/application.go
  • Line 64: warning: "ect" is a misspelling of "etc" (misspell)
  • Line 65: warning: "ect" is a misspelling of "etc" (misspell)