Sign upLogin

efritz/ij

View on GitHub
Star
config/plan.go

Summary

Maintainability
A
40 mins
Test Coverage

Method Plan.AddStage has 6 return statements (exceeds 4 allowed).
Open

func (p *Plan) AddStage(stage *Stage) error {
    if stage.BeforeStage != "" && stage.AfterStage != "" {
        return fmt.Errorf(
            "before_stage and after_stage declared in %s/%s",
            p.Name,
Severity: Major
Found in config/plan.go - About 40 mins to fix

    exported method Plan.InsertStage should have comment or be unexported
    Open

    func (p *Plan) InsertStage(stage *Stage, index int) {
    Severity: Minor
    Found in config/plan.go by golint

    exported method Plan.Merge should have comment or be unexported
    Open

    func (p *Plan) Merge(child *Plan) error {
    Severity: Minor
    Found in config/plan.go by golint

    exported type Plan should have comment or be unexported
    Open

    type Plan struct {
    Severity: Minor
    Found in config/plan.go by golint

    exported method Plan.Clone should have comment or be unexported
    Open

    func (p *Plan) Clone() *Plan {
    Severity: Minor
    Found in config/plan.go by golint

    exported method Plan.StageIndex should have comment or be unexported
    Open

    func (p *Plan) StageIndex(name string) int {
    Severity: Minor
    Found in config/plan.go by golint

    exported method Plan.AddStage should have comment or be unexported
    Open

    func (p *Plan) AddStage(stage *Stage) error {
    Severity: Minor
    Found in config/plan.go by golint

    There are no issues that match your filters.

    Category
    Status