<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Sep 25, 2015 at 3:44 PM, Ihar Hrachyshka <span dir="ltr"><<a href="mailto:ihrachys@redhat.com" target="_blank">ihrachys@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
releases are approaching, so it’s the right time to start some bike shedding on the mailing list.<br>
<br>
Recently I got pointed out several times [1][2] that I violate our commit message requirement [3] for the message lines that says: "Subsequent lines should be wrapped at 72 characters.”<br>
<br>
I agree that very long commit message lines can be bad, f.e. if they are 200+ chars. But <= 79 chars?.. Don’t think so. Especially since we have 79 chars limit for the code.<br>
<br>
We had a check for the line lengths in openstack-dev/hacking before but it was killed [4] as per openstack-dev@ discussion [5].<br>
<br>
I believe commit message lines of <=80 chars are absolutely fine and should not get -1 treatment. I propose to raise the limit for the guideline on wiki accordingly.<br></blockquote><div><br></div><div>IIUC, the lower limit for commit messages is because git displays them indented by default, which means that lines which are 80 chars long will wrap on a display which is 80 chars wide. I personally use terminal windows which are 80 chars wide, and I do find long lines in commit messages annoying, so I'm personally in favour of retaining the lower limit. Can't say I'd storm any castles if it was changed, but if most people use git the way I do[1] I guess it should stay.</div><div><br></div><div>Matt</div><div><br></div><div>[1] I have no idea if this is the case.</div></div></div></div>