runner/runner.go

Summary

Maintainability
A
3 hrs
Test Coverage

Method Runner.Run has 75 lines of code (exceeds 50 allowed). Consider refactoring.
Open

func (r *Runner) Run(plans []string) bool {
    r.logger.Info(
        nil,
        "Beginning run %s",
        r.runID,
Severity: Minor
Found in runner/runner.go - About 2 hrs to fix

    Function NewRunner has 9 arguments (exceeds 4 allowed). Consider refactoring.
    Open

        ctx context.Context,
        logger logging.Logger,
        config *config.Config,
        taskRunnerFactory TaskRunnerFactory,
        scratch *scratch.ScratchSpace,
    Severity: Major
    Found in runner/runner.go - About 1 hr to fix

      exported function NewRunner should have comment or be unexported
      Open

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

      exported type Runner should have comment or be unexported
      Open

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

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

      func (r *Runner) Run(plans []string) bool {
      Severity: Minor
      Found in runner/runner.go by golint

      There are no issues that match your filters.

      Category
      Status