<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 23, 2016 at 1:11 PM, Monty Taylor <span dir="ltr"><<a href="mailto:mordred@inaugust.com" target="_blank">mordred@inaugust.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I think this is the most important point. We have automated style tools.<br>
Humans should not be doing code review on code style. Humans should be<br>
doing code review on logic and intent and whether a design is a good<br>
idea or whether their colleague may have missed an error case or what<br>
have you.<br>
<br>
We have tools for enforcing things like tests running and code style<br>
conforming. Let the tools do their job so the humans can focus on being<br>
human.<br>
<span class="gmail-HOEnZb"></span><br></blockquote></div><br><div>I'll somewhat disagree. If code style impacts code readability then I think it is useful to review that. I don't want unreadable code to land because the automated style checkers don't complain. Basically use common sense.<br><br></div>This passes code style checks:<br><span style="font-family:monospace,monospace">    if 'f'\<br>            'e'\<br>            'e'\<br>            'l'\<br>            'i'\<br>            'n'\<br>            'g'\<br>            == \<br>            'feeling':<br>        print("hello")<br></span><br></div></div>