Branch: refs/heads/JENKINS-40370 Home: https://github.com/jenkinsci/pipeline-model-definition-plugin Commit: 289e89e07739f7cb742ceccf873d52cba8725753 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/289e89e07739f7cb742ceccf873d52cba8725753 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-01-05 (Thu, 05 Jan 2017)
Changed paths: M pipeline-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTStep.java M pipeline-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTWhen.java R pipeline-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTWhenCondition.java M pipeline-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidator.java M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidatorImpl.groovy Log Message: ----------- Get rid of ModelASTWhenCondition marker The only reason to keep it would be to keep the validateElement(ModelASTWhatever) method consistency, and frankly that's not enough. What we need is to be able to call a different validation method when a ModelASTStep is being validated within a ModelASTWhen than in other contexts, so, well, let's add validateWhenCondition(ModelASTStep) instead of relying on class magic. -- You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.