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

Support engine configuration #229

Merged
merged 1 commit into from
May 28, 2016
Merged

Conversation

mchugh19
Copy link

As discussed in #228

@mchugh19
Copy link
Author

Using jinja filters like {{ engines | yaml(False) | indent(2) }} makes this very simple. Perhaps deceptively simple?

If this style works for everyone, a bunch of manual templating work could be saved.

@herisanu
Copy link

I would say +1 to the jinja filter above. We use it a lot and it didn't cause any problems (once you find out the correct indent number).

@gravyboat
Copy link
Contributor

gravyboat commented May 28, 2016

I don't see any issue with the filter. Especially considering that it's a new addition we don't have to worry about breaking backwards compatibility. Thanks!

@gravyboat gravyboat merged commit 7f41dff into saltstack-formulas:master May 28, 2016
@gravyboat gravyboat mentioned this pull request May 28, 2016
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.

3 participants