New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIXED JENKINS-37823] Added "wrappers" section. #36
Conversation
This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation. |
🐝 |
🐝 |
|
||
pipeline { | ||
agent none | ||
wrappers { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this the name we want to go with?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Find me a better one? =)
Very nice - not 100% sold on the name "wrappers" but that is a minor thing. Can this be done at stage level too potentially? |
Conflicts: pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ModelInterpreter.groovy pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java
LGTM |
* | ||
* @author Andrew Bayer | ||
*/ | ||
public class WrappersToMapTranslator implements MethodMissingWrapper, Serializable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm starting to see a lot of similar translators, we should extract common code into something that can be inherited and reused instead of copy and paste.
🐝 |
Conflicts: pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ModelInterpreter.groovy pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/AbstractModelDefTest.java
JENKINS-37823
Trying to figure out what more tests are needed - note that this doesn't accept the
wrap
step, since it requires anode
context and we explicitly reject any step that requires anode
.cc @reviewbybees esp @stephenc @rsandell
Also cc @kuhnroyal since I know you're interested in this functionality. =)