Github user aledsage commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/242#issuecomment-59028594
LGTM. Only a couple of very minor things.
Do you want to rebase this before we commit (now that #240 is merge), or
should we just be brave and merge without (it claims it can be merged
automatically).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
