Code review comment for lp:~bcsaller/pyjuju/sane_output_test_option

Revision history for this message
Kapil Thangavelu (hazmat) wrote :

bug in base formatter.

https://codereview.appspot.com/6216053/diff/1/juju/hooks/cli.py
File juju/hooks/cli.py (right):

https://codereview.appspot.com/6216053/diff/1/juju/hooks/cli.py#newcode220
juju/hooks/cli.py:220: formatter(result, stream)
None is not a callable type, this tries a double output if formatter is
None. please default formatter to the correct default output.

https://codereview.appspot.com/6216053/

« Back to merge proposal