<p dir="ltr"><br>
On 3 Aug 2013 12:45, "Terry Reedy" <<a href="mailto:tjreedy@udel.edu">tjreedy@udel.edu</a>> wrote:<br>
><br>
><br>
><br>
> On 8/2/2013 10:26 PM, Nick Coghlan wrote:<br>
>><br>
>><br>
>> On 3 Aug 2013 11:07, "Terry Reedy" <<a href="mailto:tjreedy@udel.edu">tjreedy@udel.edu</a><br>
>> <mailto:<a href="mailto:tjreedy@udel.edu">tjreedy@udel.edu</a>>> wrote:<br>
>>  ><br>
>>  > On 8/2/2013 6:19 AM, nick.coghlan wrote:<br>
>>  ><br>
>>  >> +The Python standard library is conservative and requires limiting<br>
>>  >> +lines to 79 characters (and docstrings/comments to 72).<br>
>>  ><br>
>>  ><br>
>>  > If you (and Guido) mean that as a hard limit, then patchcheck should<br>
>> check line lengths as well as trailing whitespace.<br>
>><br>
>> That raises issues when modifying existing non-compliant files, because<br>
>> it removes the human judgement on whether a non-compliance is worth<br>
>> fixing or not.<br>
><br>
><br>
> I meant tools/scripts/patchcheck.py, not the pre-commit hook. The check would inform (especially for old files) or remind (for new files) so that judgment could be applied.</p>
<p dir="ltr">Ah, right. Yeah, that may be reasonable.</p>
<p dir="ltr">A warning option on reindent.py may be a place to start if someone wanted to implement it. Whether or not patchcheck used that option would likely depend on the initial results of running it manually :)</p>
<p dir="ltr">Cheers,<br>
Nick.<br>
><br>
> _______________________________________________<br>
> Python-checkins mailing list<br>
> <a href="mailto:Python-checkins@python.org">Python-checkins@python.org</a><br>
> <a href="http://mail.python.org/mailman/listinfo/python-checkins">http://mail.python.org/mailman/listinfo/python-checkins</a><br>
</p>