Branch: refs/heads/master Home: https://github.com/jenkinsci/pipeline-model-definition-plugin Commit: 3020057c7dabad0af4b5f5f34f4102cee274a894 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/3020057c7dabad0af4b5f5f34f4102cee274a894 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-03-21 (Tue, 21 Mar 2017)
Changed paths: M pipeline-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTArgumentList.java M pipeline-model-definition/pom.xml M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidatorImpl.groovy M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/BasicModelDefTest.java A pipeline-model-definition/src/test/resources/htmlPublisher.groovy M pom.xml Log Message: ----------- [FIXED JENKINS-41456] Properly validate publishHTML Specifically, when we see that there's a sole required parameter, the only provided argument by the user is a map, at least one of those map keys doesn't have a corresponding parameter in the describable, and the sole required parameter's type is a describable itself, validate the argument map against that parameter type describable instead of against the parent describable. So for publishHTML, that means realizing `target` is an `HtmlPublisherTarget` and validating the map against that. Commit: 3ac633a2856088af7e51146590e78fef1bee13ea https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/3ac633a2856088af7e51146590e78fef1bee13ea Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-03-22 (Wed, 22 Mar 2017) Changed paths: M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidatorImpl.groovy Log Message: ----------- Clarify return statements Commit: fd0759ac82c32c13d1c6feb52ebec13b000589c7 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/fd0759ac82c32c13d1c6feb52ebec13b000589c7 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-03-27 (Mon, 27 Mar 2017) Changed paths: M pipeline-model-api/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTArgumentList.java M pipeline-model-definition/pom.xml M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidatorImpl.groovy M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/BasicModelDefTest.java A pipeline-model-definition/src/test/resources/htmlPublisher.groovy M pom.xml Log Message: ----------- Merge pull request #142 from abayer/jenkins-41456 [FIXED JENKINS-41456] Properly validate publishHTML Compare: https://github.com/jenkinsci/pipeline-model-definition-plugin/compare/562bb499344c...fd0759ac82c3 -- 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.