A formatter bot would be quite complicated to introduce without disrupitions of everybody's workflow (remember that we have about half a million lines of C code in the Python repo). If you want to discuss that please start a new thread on python-dev.
On Tue, Jan 19, 2016 at 12:22 PM, francismb <franci...@email.de> wrote: > Hi Brett, > > On 01/19/2016 12:20 AM, Brett Cannon wrote: > > On Sun, 17 Jan 2016 at 11:10 Brett Cannon <br...@python.org> wrote: > > > >> While doing a review of http://bugs.python.org/review/26129/ I asked to > >> have curly braces put around all `if` statement bodies. Serhiy pointed > out > >> that PEP 7 says curly braces are optional: > >> https://www.python.org/dev/peps/pep-0007/#id5. I would like to change > >> that. > >> > >> My argument is to require them to prevent bugs like the one Apple made > >> with OpenSSL about two years ago: > >> https://www.imperialviolet.org/2014/02/22/applebug.html. Skipping the > >> curly braces is purely an aesthetic thing while leaving them out can > lead > >> to actual bugs. > >> > >> Anyone object if I update PEP 7 to remove the optionality of curly > braces > >> in PEP 7? > >> > > > > What about about a code formatter bot ? (new workflow). If one just > could agree, then those reviews should just disappear (?). > > Regards, > francis > _______________________________________________ > Python-Dev mailing list > Python-Dev@python.org > https://mail.python.org/mailman/listinfo/python-dev > Unsubscribe: > https://mail.python.org/mailman/options/python-dev/guido%40python.org > -- --Guido van Rossum (python.org/~guido)
_______________________________________________ Python-Dev mailing list Python-Dev@python.org https://mail.python.org/mailman/listinfo/python-dev Unsubscribe: https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com