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

Added in ability for user to redefine update center plugin URL #134

Closed
wants to merge 1 commit into from

Conversation

carlpulley
Copy link
Contributor

Jenkins update centre can be unreliable with some plugin downloads. Hence we here add in the ability for a user to redefine the update centre plugin URL.

…releasing us from jenkins update center dependency)
@jenkinsadmin
Copy link

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

@jlambert121
Copy link
Contributor

@carlpulley are you able to write some tests around this parameter? Other than that, LGTM.

Looks like #130 also includes changes for this, but in a slightly different manner.

@rtyler rtyler added this to the 1.3.0 - Bernard milestone Sep 26, 2014
@rtyler
Copy link

rtyler commented Sep 26, 2014

I've started refactoring this a bit locally, will update with a new pull request when I have it.

Basically adding tests and setting some default for the value in jenkins::params

@rtyler
Copy link

rtyler commented Sep 27, 2014

Moving this work into #190

@rtyler rtyler closed this Sep 27, 2014
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