Branch: refs/heads/master Home: https://github.com/jenkinsci/pipeline-model-definition-plugin Commit: d533dd551dadca3d278f3c877aead9016d514b8c https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/d533dd551dadca3d278f3c877aead9016d514b8c Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-11-20 (Mon, 20 Nov 2017)
Changed paths: M pipeline-model-definition/pom.xml M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Aborted.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Changed.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Failure.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/NotBuilt.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Success.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Unstable.groovy M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/BuildConditionResponderTest.java A pipeline-model-definition/src/test/resources/contextResultOverridesRunResult.groovy A pipeline-model-definition/src/test/resources/junitResult.xml M pipeline-model-extensions/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/model/BuildCondition.java M pom.xml Log Message: ----------- [FIXED JENKINS-37663] Check execution result in post as well The new junit step doesn't set the Run to UNSTABLE directly, it sets the context (and therefore the execution) result to UNSTABLE. We should be checking the execution result as well as the Run result. Commit: 42a4c3f1092429788c9854a6afa0f942d52e0082 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/42a4c3f1092429788c9854a6afa0f942d52e0082 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-11-21 (Tue, 21 Nov 2017) Changed paths: M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Aborted.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Changed.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Failure.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/NotBuilt.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Unstable.groovy Log Message: ----------- Code review comments cleaning up conditions Commit: ac7d419c68406deb2325a2b29d4c41c4faab51f2 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/ac7d419c68406deb2325a2b29d4c41c4faab51f2 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-11-21 (Tue, 21 Nov 2017) Changed paths: M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Changed.groovy Log Message: ----------- We actually do need to check whether the current result is null... Commit: e396a2a61bcc0072f956023ec02e1bfcc77f3763 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/e396a2a61bcc0072f956023ec02e1bfcc77f3763 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-11-21 (Tue, 21 Nov 2017) Changed paths: M pipeline-model-definition/pom.xml M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Aborted.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Changed.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Failure.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/NotBuilt.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Success.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/conditions/Unstable.groovy M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/BuildConditionResponderTest.java A pipeline-model-definition/src/test/resources/contextResultOverridesRunResult.groovy A pipeline-model-definition/src/test/resources/junitResult.xml M pipeline-model-extensions/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/model/BuildCondition.java M pom.xml Log Message: ----------- Merge pull request #220 from abayer/jenkins-37663 [FIXED JENKINS-37663] Check execution result in post as well Compare: https://github.com/jenkinsci/pipeline-model-definition-plugin/compare/91011f0f6d6b...e396a2a61bcc -- 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.