Code review comment for lp://qastaging/~bladernr/checkbox/1257308

Revision history for this message
Jeff Lane  (bladernr) wrote :

Yes... without the newlines (note the preceding newline above it) all
the output was wedged together and made it more difficult to my eyes
to pick out the failure and quickly parse the output visually.

I can remove those if you prefer, it was just a bit of extra
formatting I threw in (and forgot to mention in the MR).

On Tue, Dec 3, 2013 at 11:05 AM, Zygmunt Krynicki
<email address hidden> wrote:
> Review: Needs Information
>
> 37 - print("theoretical max %sMb/s" % int(self.iface.max_speed))
> 38 + print("theoretical max %sMb/s\n" % int(self.iface.max_speed))
>
> Is this intended?
> --
> https://code.launchpad.net/~bladernr/checkbox/1257308/+merge/197554
> You are the owner of lp:~bladernr/checkbox/1257308.

--
Jeff Lane - Server and Cloud Certification and Tools Development
Ubuntu Ham: W4KDH
Freenode IRC: bladernr or bladernr_
gpg: 1024D/3A14B2DD 8C88 B076 0DD7 B404 1417 C466 4ABD 3635 3A14 B2DD

« Back to merge proposal