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

Implement methods for adding and removing host configurations #5

Merged
merged 1 commit into from
May 19, 2015

Conversation

rbywater
Copy link
Member

This commit adds the ability to programatically add and remove host
configurations.

This commit adds the ability to programatically add and remove host
configurations.
@cloudbees-pull-request-builder

plugins » publish-over-ssh-plugin #9 SUCCESS
This pull request looks good

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@eddyhub
Copy link

eddyhub commented May 18, 2015

Hi, I was wondering if anyone had a chance to take a look at this? Is there a chance this commit will be merged any time soon?

@rbywater
Copy link
Member Author

I've asked on the mailing list a while back and got no response. So, unless someone else jumps in first, I'll look at doing at merging this PR and doing a new release soon myself.

@eddyhub
Copy link

eddyhub commented May 18, 2015

I need this commit for a puppet module and was about to add the necessary methods myself but a check of the pull requests revealed, that you already did the necessary changes. So if I can help getting the changes to a new release just tell me.

rbywater added a commit that referenced this pull request May 19, 2015
Implement methods for adding and removing host configurations
@rbywater rbywater merged commit c8cb386 into jenkinsci:master May 19, 2015
@rbywater rbywater deleted the addremovehosts branch May 19, 2015 08:17
@rbywater
Copy link
Member Author

Just released 1.13 so hopefully that will flow through soon with this change in. (Note I had a few issues so hopefully it does come through properly!)

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

Successfully merging this pull request may close these issues.

4 participants