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

Using #to_yaml instead of printing each element #410

Merged
merged 3 commits into from
Feb 23, 2018

Conversation

danzilio
Copy link
Member

This PR modifies the template for r10k.yaml to use the #to_yaml
method to generate valid YAML data structures from the data passed to
the template. Previously, we'd print each element assuming there would
be no nested data structures. This PR allows nested data structures to
be passed to git_settings, forge_settings, and deploy_settings
parameters.

This PR modifies the template for `r10k.yaml` to use the `#to_yaml`
method to generate valid YAML data structures from the data passed to
the template. Previously, we'd print each element assuming there would
be no nested data structures. This PR allows nested data structures to
be passed to `git_settings`, `forge_settings`, and `deploy_settings`
parameters.

DRYing out my code
@bastelfreak bastelfreak added the enhancement New feature or request label Feb 23, 2018
@bastelfreak
Copy link
Member

Thanks for the PR @danzilio !

@bastelfreak bastelfreak merged commit a91ad6b into voxpupuli:master Feb 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants