On Fri, Mar 27, 2020 at 12:51:23PM +0100, Andrea Bolognani wrote:
On Fri, 2020-03-27 at 10:53 +0000, Daniel P. Berrangé wrote:
On Thu, Mar 26, 2020 at 06:06:37PM +0100, Andrea Bolognani wrote:
On Thu, 2020-03-26 at 12:35 +0000, Daniel P. Berrangé wrote:
+ if "localhost" in line: + print(" ❌ FAIL: bad email in %s" % line) + errors = True
... but if you absolutely must have them, at least don't try to mess with indentation - aligning text and emoji is basically never going to work reliably anyway. Please consider applying the diff at the end of this message if you think dropping the emoji is not an option.
I wasn't actually trying to align them - the fail lines are intentionally indented, though not by enough.
Then indent them by 4 spaces or more. Anything less than that will just look bad.
Yep, will do. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|