-
-
Notifications
You must be signed in to change notification settings - Fork 352
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
make elb/policies collection standalone #128
Conversation
* there is no need to inject data from the load balancer model when an index request exists that maps directly
b2b46d0
to
cdd724d
Compare
def load_balancer | ||
collection.load_balancer | ||
service.load_balancers.new(identity: collection.load_balancer_id) |
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.
Could you fix the 1.8 incompatible named argument here? Hopefully we'll reach a point where we can drop 1.8 soon, but not quite ready for the major/breaking change I'm afraid. Thanks!
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.
gets me every time.
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.
Yeah, stuff like this is starting to bite us more and more often. Breaking changes just also are, well, breaking, so caution prevails. Would love to cut it out soon though.
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.
it's the last "," before the end of a Hash definition that is perhaps the biggest annoyance. Quite possible the most understated Ruby improvement.
* fix 1.9 only syntax * don't rely on possible collection for a required attribute
* pull load_balancer_id from the collection and set on the model
make elb/policies collection standalone
Thanks! |
an index request exists that maps directly