Code review comment for lp://staging/~benji/landscape-client/better-self-signed-cert-ux-3

Revision history for this message
Benji York (benji) wrote :

> Sorry for being a pain :) But the latest diff isn't what I would expect it to
> be.

I pushed, but apparently forgot to commit beforehand.

> BTW, maybe the patch was odd because you looked at the wrong pastebin? ;)

This is the pastebin I was referring to: https://pastebin.canonical.com/126660/

The oddities I see are the blank lines with a dot in the indicator position and the lack of whitespace after the file name in the headers.

« Back to merge proposal