<div dir="ltr">Thank you all for responding and sharing your thoughts. It seems that although the longer limit might be nice it will bring more complications than we want to. For that reason we can consider this discussion as closed.  That is, there will be no chances.<div><br></div><div>Thanks again</div><div>Miro</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 31, 2022 at 8:22 AM Sylvain Bauza <<a href="mailto:sbauza@redhat.com">sbauza@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le sam. 28 mai 2022 à 04:36, Miro Tomaska <<a href="mailto:mtomaska@redhat.com" target="_blank">mtomaska@redhat.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hello All,<div><br></div><div>This is probably going to be a hot topic but I was wondering if the community ever considered raising the default 79 characters line limit. I have seen some places where even a very innocent line of code needs to be split into two lines. I have also seen some code where I feel like variable names were abbreviated on purpose to squeeze everything into one line.</div><div><br></div><div>How does the community feel about raising the E501 limit to 119 characters? The 119 character limit is the second most popular limit besides the default one. It's long enough to give a developer enough room for descriptive variables without being forced to break lines too much. And it is short enough for a diff between two files to look OK.</div><div><br></div><div>The only downside I can see right now is that it's not easy to convert an existing code. So we will end up with files where the new code is 79+ characters and the "old" code is <=79. I can also see an argument where someone might have trouble reviewing a patch on a laptop screen (assuming standard 14" screen) ?<br></div><div><br></div><div>Here is an example of one extreme, a diff of two files maxing out at 119 characters</div><div><br></div><div><a href="https://review.opendev.org/c/opendev/sandbox/+/843697/1..2" target="_blank">https://review.opendev.org/c/opendev/sandbox/+/843697/1..2</a><br></div><div> </div><div>Thank you for your time and I am looking forward to this conversation :)</div><div> <br clear="all"></div></div></blockquote><div><br></div><div>My personal opinion is while I understand the reasoning behind, my concern is about the community support. We don't have a lot of contributors at the moment, and if we would like to modify the default limit, I'm pretty sure it would take a lot of time for folks at least reviewing some changes that are not really priorities...</div><div><br></div><div>Also, as said by some folks, changing the default would also create some problems for some of our contributors and eventually, if some projects would accept 119 characters while some others not,  it would also be a new community difference between projects... (and yet again some new nit that new contributors need to understand)</div><div><br></div><div>Sorry, -1 for those reasons.</div><div><br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div>--<br></div><div dir="ltr"><div dir="ltr"><font face="monospace" color="#999999">Miro Tomaska</font></div><div dir="ltr"><div><font face="monospace" color="#999999">irc: mtomaska</font></div><div><font face="monospace" color="#999999">Red Hat</font></div></div></div></div></div>
</blockquote></div></div>
</blockquote></div>