Code review comment for lp:~stub/charm-helpers/hook-magic

Revision history for this message
James Page (james-page) wrote :

Hi Stuart

Looks good - aside from one minor lint issue:

tests/core/test_hookenv.py:712:9: E301 expected 1 blank line, found 0

However its more work for you to update that and resubmit than me just to fix it on merge.

+1

review: Approve

« Back to merge proposal