pkg/kepval: clarify prr required check

Make it more obvious that PRR is only getting checked when status is
implementable.  Don't return early, so that missing data warnings start
to show through

Author: spiffxp

pkg/kepval/approval.go

@@ -100,15 +100,10 @@ func ValidatePRR(kep *api.Proposal, h *api.PRRHandler, prrDir string) error {
 func isPRRRequired(kep *api.Proposal) (required, missingMilestone, missingStage bool, err error) {
 	logrus.Debug("checking if PRR is required")
 
-	required = true
+	required = kep.Status == api.ImplementableStatus
 	missingMilestone = kep.IsMissingMilestone()
 	missingStage = kep.IsMissingStage()
 
-	if kep.Status != api.ImplementableStatus {
-		required = false
-		return required, missingMilestone, missingStage, nil
-	}
-
 	if missingMilestone {
 		logrus.Warnf("Missing the latest milestone field: %s", kep.Filename)
 		return required, missingMilestone, missingStage, nil