Code review comment for lp:~adam-collard/charm-helpers/amulet-get-process-id-list-fixes

Revision history for this message
Chris Glass (tribaal) wrote :

Thanks for your branch. On principle it looks good, but some tests fail and others clutter stdout/stderr on "make test".

I'm going to have to mark this as need fixing until "make test" produces a clean (and passing) output.

review: Needs Fixing

« Back to merge proposal