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

Give default value to config from facter #189

Merged
merged 1 commit into from Jun 10, 2018

Conversation

FooBarQuaxx
Copy link
Contributor

This PR will fix the hiera hierarchy values in case the platform and/or the target is not set from the command line the thing that will prevent the documented lookup to function correctly.

Copy link
Owner

@bernd bernd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! 👍

@bernd bernd merged commit e899941 into bernd:master Jun 10, 2018
bernd added a commit that referenced this pull request Jun 1, 2019
Changes:

* Use factor defaults for `platform` and `target` values. (FooBarQuaxx / #189)
* Add `:username` and `:password` parameters to the SVN source handler. (tomeon / #191)
* Add `Recipe#extract` method to allow custom extraction logic.
* Fix tmux location in getting started guide. (cvhbsk / #195)
* Use RPM target for SLES platform. (FooBarQuaxx / #198)
* Add `targets` block for target specific setting in recipe. (FooBarQuaxx / #199)
* Ensure `:noop` sources are always treated as fetchable. (tomeon / #200)
* Allow usage of newer puppet versions. (lukasz-e / #201)
* Add sha512 checksum support. (b00ga / #202)
* Remove duplicate require. (FooBarQuaxx / #204)
* Automatically gunzip gzipped patch files. (davewongillies / #208)
* Add a golang util. (davewongillies / #209)
suledani97 pushed a commit to realCity/fpm-cookery that referenced this pull request Jul 30, 2020
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.

None yet

2 participants