Branch: refs/heads/master Home: https://github.com/jenkinsci/pipeline-model-definition-plugin Commit: f533ef99e31ce44cbe999653e538ea39fdf772dc https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/f533ef99e31ce44cbe999653e538ea39fdf772dc Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-05 (Wed, 05 Oct 2016)
Changed paths: M pom.xml M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTBuildParameter.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTBuildParameters.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTElement.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTJobProperties.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTJobProperty.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTKeyValueOrMethodCallPair.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodArg.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodCallFromArguments.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTPipelineDef.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTPositionalArgumentList.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTTrigger.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTTriggers.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTValue.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/JobProperties.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/MethodsToList.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Parameters.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Root.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Tools.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Triggers.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/JSONParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidator.groovy M src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ModelStepLoader.java M src/main/resources/ast-schema.json M src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ClosureModelTranslator.groovy A src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/MethodsToListTranslator.groovy M src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ModelInterpreter.groovy M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/JobPropertiesTest.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ParametersTest.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/TriggersTest.java A src/test/resources/json/simpleJobProperties.json A src/test/resources/json/simpleParameters.json A src/test/resources/json/simpleTriggers.json A src/test/resources/simpleJobProperties.groovy A src/test/resources/simpleParameters.groovy A src/test/resources/simpleTriggers.groovy Log Message: ----------- [FIXED JENKINS-37824] Adding job property-related functionality More specifically, added "jobProperties", "parameters" and "triggers" section and related functionality. This required some fairly complex logic for translating something like "buildDiscarder(logRotator(numToKeepStr:'1'))" in the config into something that can do what we need at runtime. Hooray complexity! Commit: 2e263edd17b470b72ad5591eb11a5f1d8e6c9fac https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/2e263edd17b470b72ad5591eb11a5f1d8e6c9fac Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-05 (Wed, 05 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTBuildParameter.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTBuildParameters.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTJobProperties.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTJobProperty.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTKeyValueOrMethodCallPair.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodArg.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodCall.groovy R src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodCallFromArguments.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTTrigger.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTTriggers.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/JobProperties.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/MethodsToList.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Parameters.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Triggers.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/JSONParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidator.groovy M src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/MethodsToListTranslator.groovy Log Message: ----------- Cleanup and javadoc Commit: fe3b93b43119df3621025e02a428f7d7f5c4aed4 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/fe3b93b43119df3621025e02a428f7d7f5c4aed4 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-05 (Wed, 05 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTBuildParameters.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTJobProperties.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTTriggers.groovy Log Message: ----------- Fix Groovy generation typoes Commit: db51bc15862c3bc9355eae7896455b679ec834ce https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/db51bc15862c3bc9355eae7896455b679ec834ce Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-05 (Wed, 05 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/JobProperties.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Triggers.groovy Log Message: ----------- Fixing findbugs errors Commit: 07a884107247d6157d9a73245ca0e0f546727eca https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/07a884107247d6157d9a73245ca0e0f546727eca Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-05 (Wed, 05 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/JobProperties.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Triggers.groovy Log Message: ----------- Fix further findbugs errors Commit: 636d7ceae5d48162b3f7a9f85ca971bfad93b7cc https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/636d7ceae5d48162b3f7a9f85ca971bfad93b7cc Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-05 (Wed, 05 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodCall.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTStep.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/JSONParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidator.groovy M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ValidatorTest.java A src/test/resources/errors/blockInJobProperties.groovy A src/test/resources/errors/blockInParameters.groovy A src/test/resources/errors/blockInTriggers.groovy A src/test/resources/errors/closureAsMethodCallArg.groovy A src/test/resources/errors/emptyJobProperties.groovy A src/test/resources/errors/emptyParameters.groovy A src/test/resources/errors/emptyTriggers.groovy A src/test/resources/errors/invalidParameterTypeMethodCall.groovy A src/test/resources/errors/mixedMethodArgs.groovy A src/test/resources/errors/rejectPropertiesStepInMethodCall.groovy A src/test/resources/errors/tooFewMethodCallArgs.groovy A src/test/resources/errors/wrongParameterNameMethodCall.groovy A src/test/resources/json/errors/emptyJobProperties.json A src/test/resources/json/errors/emptyParameters.json A src/test/resources/json/errors/emptyTriggers.json A src/test/resources/json/errors/invalidParameterTypeMethodCall.json A src/test/resources/json/errors/missingRequiredMethodCallArg.json A src/test/resources/json/errors/mixedMethodArgs.json A src/test/resources/json/errors/rejectPropertiesStepInMethodCall.json A src/test/resources/json/errors/wrongParameterNamedMethodCall.json Log Message: ----------- A bunch of testing of invalid syntax etc for method calls and the like. Commit: 375862bd4d6907d2841cf41de89ed9757d7bb1f2 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/375862bd4d6907d2841cf41de89ed9757d7bb1f2 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-06 (Thu, 06 Oct 2016) Changed paths: M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java A src/test/resources/json/errors/wrongParameterNameMethodCall.json R src/test/resources/json/errors/wrongParameterNamedMethodCall.json Log Message: ----------- Fixing some failing tests Commit: 9802ef7bc5c7f082e4ceccbeb87558561abdb9e5 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/9802ef7bc5c7f082e4ceccbeb87558561abdb9e5 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-06 (Thu, 06 Oct 2016) Changed paths: M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java Log Message: ----------- Actually fixing those tests. Sigh. Commit: 15816412f69ba1d8ae0aec33ed09951185744109 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/15816412f69ba1d8ae0aec33ed09951185744109 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-10 (Mon, 10 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/Converter.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/endpoints/ModelConverterAction.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/endpoints/ModelConverterActionStepsTest.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/util/IsJsonObjectContaining.java A src/test/resources/json/steps/arrayEcho.json A src/test/resources/json/steps/simpleEcho.json A src/test/resources/json/steps/simpleScript.json Log Message: ----------- Merge branch 'master' into jenkins-37824 Commit: 3b0271d4ac42d4343b72cda5ec2394910a165198 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/3b0271d4ac42d4343b72cda5ec2394910a165198 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-10 (Mon, 10 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/JobProperties.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Tools.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidator.groovy M src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/MethodsToListTranslator.groovy M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/JobPropertiesTest.java M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ValidatorTest.java A src/test/resources/errors/rejectMapsForTriggerDefinition.groovy A src/test/resources/multipleProperties.groovy Log Message: ----------- Testing more combinations. * Fixed handling of no-arg method calls at runtime. * Properly error out on legacy metastep syntax for job properties et al - couldn't find a viable way to support both syntaxes without waaaay too much pain, so, we just support symbols. * Removed non-symbol "did you mean...?" and validation keys for props/params/triggers. * Made sure we were properly excluding pipelineTriggers and parameters from job properties! * Avoided duplicate error messages when trig/prop/param has compilation-time errors - no need to have a second validation error on top of that. Commit: aecca5e630947964d3f008eb9ed54defe3e7e29c https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/aecca5e630947964d3f008eb9ed54defe3e7e29c Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-10 (Mon, 10 Oct 2016) Changed paths: M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTPipelineDef.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTValue.groovy M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java A src/test/resources/json/stringsNeedingEscapeLogic.json A src/test/resources/stringsNeedingEscapeLogic.groovy Log Message: ----------- Merge branch 'master' into jenkins-37824 Conflicts: src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java Commit: 7b150a14952e8daf333c442aac3dcfae0db012d2 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/7b150a14952e8daf333c442aac3dcfae0db012d2 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-10 (Mon, 10 Oct 2016) Changed paths: M pom.xml Log Message: ----------- Merge branch 'master' into jenkins-37824 Commit: f56fb9f2e51103dca8a4b9455eadc7dc73736386 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/f56fb9f2e51103dca8a4b9455eadc7dc73736386 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-10 (Mon, 10 Oct 2016) Changed paths: M SYNTAX.md M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy Log Message: ----------- Updating SYNTAX.md and removing dead comment. Commit: b06274bc66a812c3e643f3ba517fc07a7f69f4ba https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/b06274bc66a812c3e643f3ba517fc07a7f69f4ba Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2016-10-11 (Tue, 11 Oct 2016) Changed paths: M SYNTAX.md M pom.xml M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTBuildParameter.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTBuildParameters.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTElement.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTJobProperties.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTJobProperty.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTKeyValueOrMethodCallPair.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodArg.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTMethodCall.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTPipelineDef.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTPositionalArgumentList.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTStep.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTTrigger.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTTriggers.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/ast/ModelASTValue.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/JobProperties.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/MethodsToList.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Parameters.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Root.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Tools.groovy A src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Triggers.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/JSONParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/parser/ModelParser.groovy M src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/validator/ModelValidator.groovy M src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/ModelStepLoader.java M src/main/resources/ast-schema.json M src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ClosureModelTranslator.groovy A src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/MethodsToListTranslator.groovy M src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ModelInterpreter.groovy M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/JobPropertiesTest.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ParametersTest.java A src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/TriggersTest.java M src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ValidatorTest.java A src/test/resources/errors/blockInJobProperties.groovy A src/test/resources/errors/blockInParameters.groovy A src/test/resources/errors/blockInTriggers.groovy A src/test/resources/errors/closureAsMethodCallArg.groovy A src/test/resources/errors/emptyJobProperties.groovy A src/test/resources/errors/emptyParameters.groovy A src/test/resources/errors/emptyTriggers.groovy A src/test/resources/errors/invalidParameterTypeMethodCall.groovy A src/test/resources/errors/mixedMethodArgs.groovy A src/test/resources/errors/rejectMapsForTriggerDefinition.groovy A src/test/resources/errors/rejectPropertiesStepInMethodCall.groovy A src/test/resources/errors/tooFewMethodCallArgs.groovy A src/test/resources/errors/wrongParameterNameMethodCall.groovy A src/test/resources/json/errors/emptyJobProperties.json A src/test/resources/json/errors/emptyParameters.json A src/test/resources/json/errors/emptyTriggers.json A src/test/resources/json/errors/invalidParameterTypeMethodCall.json A src/test/resources/json/errors/missingRequiredMethodCallArg.json A src/test/resources/json/errors/mixedMethodArgs.json A src/test/resources/json/errors/rejectPropertiesStepInMethodCall.json A src/test/resources/json/errors/wrongParameterNameMethodCall.json A src/test/resources/json/simpleJobProperties.json A src/test/resources/json/simpleParameters.json A src/test/resources/json/simpleTriggers.json A src/test/resources/multipleProperties.groovy A src/test/resources/simpleJobProperties.groovy A src/test/resources/simpleParameters.groovy A src/test/resources/simpleTriggers.groovy Log Message: ----------- Merge pull request #27 from abayer/jenkins-37824 [FIXED JENKINS-37824] Adding job property-related functionality Compare: https://github.com/jenkinsci/pipeline-model-definition-plugin/compare/1b04fe7d9e7f...b06274bc66a8 -- 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.