Re: [libvirt PATCH v3 11/12] gitlab: introduce a check for validate DCO sign-off

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 :|





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux