Code review comment for lp:~michael.nelson/charm-helpers/ansible-support

Revision history for this message
Matthew Wedgwood (mew) wrote :

Hey, Michael,

This looks awesome, thanks! I can't wait to try it out.

Regarding juju_state_to_yaml(), I'd say it belongs in core.hookenv. Bonus points for including a larger chunk of execution_environment().

-Matthew

review: Approve

« Back to merge proposal