-
Notifications
You must be signed in to change notification settings - Fork 11
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 tags in template and stack config #23
Conversation
I've made stack_config INHERITABLE_PROPERTIES now override template_config .. I added a dependency on hashie for deep_merge as it's used in the specs in a similar way. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one minor nit, but otherwise looks great!
lib/bora/stack.rb
Outdated
# Expand Tags to key, value pairs | ||
if @stack_config[:tags] | ||
@stack_config[:tags] = @stack_config[:tags].collect { |k, v| { key: k, value: v } } | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mutating @stack_config
doesn't feel quite right here. Creates up side effects, for example what if this method happened to be called twice? Maybe assign the result of the "select" on the next line below to a temporary var, and expand the tags on that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK - Will do.
Updated! |
Released as 1.6.0 |
Sorry - pry as a development dependency slipped in. |
As per #22 an initial implementation of tags support for stack creation.