Juju Charms Collection
juju-gui package

Code review comment for lp://staging/~frankban/charms/precise/juju-gui/bug-1132905-log-hook

Revision history for this message

Gary Poster (gary) wrote on 2013-02-25:

#

LGTM.

Nice tests.

Thanks!

https://codereview.appspot.com/7379050/diff/1/hooks/config-changed
File hooks/config-changed (right):

https://codereview.appspot.com/7379050/diff/1/hooks/config-changed#newcode135
hooks/config-changed:135: with log_hook():
Nice!

https://codereview.appspot.com/7379050/

« Back to merge proposal