Code review comment for lp:~radix/subunit/report-errors-better

Revision history for this message
Robert Collins (lifeless) wrote :

Ok, I've looked into this. We use splitlines(), this strips \n's, so its appropriate to be adding them back. The reason we split is to allow escaping each line to prevent false closing ']' - even though that isn't currently hooked in :(.

So - thanks for the patch, but I won't be merging it.

review: Disapprove

« Back to merge proposal