Skip to content

Commit

Permalink
Make PRR metadata required
Browse files Browse the repository at this point in the history
  • Loading branch information
ehashman committed May 4, 2021
1 parent 29a891b commit 724e7e9
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions pkg/kepval/approval.go
Original file line number Diff line number Diff line change
Expand Up @@ -105,20 +105,17 @@ func isPRRRequired(kep *api.Proposal) (required, missingMilestone, missingStage
missingStage = kep.IsMissingStage()

if kep.Status != "implementable" {
required = false
return required, missingMilestone, missingStage, nil
}

if missingMilestone {
required = false
// TODO: Make this a specialized error
logrus.Warnf("KEP %s is missing the latest milestone field. This will become a validation error in future releases.", kep.Number)

return required, missingMilestone, missingStage, nil
}

if missingStage {
required = false
// TODO: Make this a specialized error
logrus.Warnf("KEP %s is missing the stage field. This will become a validation error in future releases.", kep.Number)

Expand Down

0 comments on commit 724e7e9

Please sign in to comment.