Sign upLogin

efritz/ij

View on GitHub
Star
runner/plan.go

Summary

Maintainability
A
2 hrs
Test Coverage

Method PlanRunner.Run has 63 lines of code (exceeds 50 allowed). Consider refactoring.
Open

func (r *PlanRunner) Run(
    context *RunContext,
    name string,
    prefix *logging.Prefix,
) bool {
Severity: Minor
Found in runner/plan.go - About 1 hr to fix

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

        ctx context.Context,
    config *config.Config,
    taskRunnerFactory TaskRunnerFactory,
    logger logging.Logger,
    env []string,
    Severity: Minor
    Found in runner/plan.go - About 35 mins to fix

      exported method PlanRunner.ShouldRun should have comment or be unexported
      Open

      func (r *PlanRunner) ShouldRun(context *RunContext, name string) bool {
      Severity: Minor
      Found in runner/plan.go by golint

      exported function NewPlanRunner should have comment or be unexported
      Open

      func NewPlanRunner(
      Severity: Minor
      Found in runner/plan.go by golint

      exported method PlanRunner.Run should have comment or be unexported
      Open

      func (r *PlanRunner) Run(
      Severity: Minor
      Found in runner/plan.go by golint

      exported method PlanRunner.IsDisabled should have comment or be unexported
      Open

      func (r *PlanRunner) IsDisabled(context *RunContext, name string) (bool, error) {
      Severity: Minor
      Found in runner/plan.go by golint

      exported type PlanRunner should have comment or be unexported
      Open

      type PlanRunner struct {
      Severity: Minor
      Found in runner/plan.go by golint

      There are no issues that match your filters.

      Category
      Status