Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

enable install of cli_helper when jenkins::cli => true #488

Merged
merged 2 commits into from
Feb 8, 2016

Conversation

jhoblitt
Copy link
Member

@jhoblitt jhoblitt commented Feb 3, 2016

  • make true the default value of the jenkins class cli param

This modules uses the cli jar + groovy helper for many of its'
management functions and is currently relying on these to be
automatically included by dependent resources. For most use cases of
this module, the cli jar + groovy helper are automatically installed
regardless of the value of the cli param. However, this does not
work for the experimental native types which are dependent on the cli
jar and groovy helper.

+ make true the default value of the `jenkins` class `cli` param

This modules uses the cli jar + groovy helper for many of its'
management functions and is currently relying on these to be
automatically included by dependent resources.  For most use cases of
this module, the cli jar + groovy helper are automatically installed
regardless of the value of the `cli` param.  However, this does not
work for the experimental native types which are dependent on the cli
jar and groovy helper.
@jenkinsadmin
Copy link

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

@jhoblitt jhoblitt added this to the 1.7.0 - Oddjob milestone Feb 8, 2016
rtyler pushed a commit that referenced this pull request Feb 8, 2016
enable install of cli_helper when jenkins::cli => true
@rtyler rtyler merged commit 48397cd into voxpupuli:master Feb 8, 2016
@jhoblitt jhoblitt deleted the feature/install-cli-by-default branch April 15, 2016 20:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants