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

feat(pillar): some config values can be treated as lists #15

Merged
merged 2 commits into from
Oct 8, 2019
Merged

feat(pillar): some config values can be treated as lists #15

merged 2 commits into from
Oct 8, 2019

Conversation

n-rodriguez
Copy link
Member

Fix #14

@n-rodriguez n-rodriguez requested review from myii and vutny October 6, 2019 15:25
Copy link

@vutny vutny left a comment

Choose a reason for hiding this comment

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

Awesome, @n-rodriguez . I love the join_if_list macro!
There is a small suggestion below, but you got my approval.

@n-rodriguez
Copy link
Member Author

n-rodriguez commented Oct 7, 2019

@aboe76
Copy link
Member

aboe76 commented Oct 7, 2019

maybe @jerrykan would like to help too?

@n-rodriguez n-rodriguez merged commit 66150d4 into saltstack-formulas:master Oct 8, 2019
@n-rodriguez n-rodriguez deleted the wip/lists branch October 8, 2019 18:33
@saltstack-formulas-travis

🎉 This PR is included in version 0.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Treat other_hostnames and local_interfaces as list
4 participants