Skip to content
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

This PR fixes JENKINS-32987, it completes PR #68 #70

Merged
merged 2 commits into from Sep 20, 2016

Conversation

aheritier
Copy link
Member

No description provided.

@aheritier
Copy link
Member Author

Can you have a look at it @reviewbybees @lacostej please ?

@ghost
Copy link

ghost commented Jul 25, 2016

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.

@aheritier
Copy link
Member Author

@madmunki
Copy link

Can we pull #69 as well? it was a single line of code that changed. I have been using the snapshot for it in our production build machine for awhile with no issues.

@patcadelina
Copy link

LGTM @aheritier, by all means let's merge. Thanks!

@daspilker
Copy link
Member

Any progress on this? If this has been released, I will merge jenkinsci/job-dsl-plugin#750.

@aheritier aheritier merged commit 3cf6865 into jenkinsci:master Sep 20, 2016
@aheritier
Copy link
Member Author

@daspilker Merged. I will release it in few minutes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants