Code review comment for lp:~fwereade/pyjuju/add-upstart-class

Revision history for this message
William Reade (fwereade) wrote :

[0,1]

Cool, thanks

[2,3]

...gaah, missing @inlineCallbacks~s led to some untestedness; root_path tested explicitly. After consultation with bcsaller (original author), it's fine to drop the "start on filesystem or" complication, so it's gone.

« Back to merge proposal