There&#x27;s a line to walk for sure. When I code review a line that looks like this:<p><pre><code>  $foo=fn1($bar) &amp;&amp;$baz = fn2 ($qux)
</code></pre>
I always send it back. Do I look pedantic? Probably, but to me consistency is important. It&#x27;s about <i>caring</i> about what you&#x27;re doing. If I see a line like this, I assume the person who wrote it doesn&#x27;t pay a lot of attention to detail and I pay closer attention to the rest of their commit. I mean, if you can&#x27;t be troubled to set your IDE preferences to the team&#x2F;project&#x27;s rules, what&#x27;s that say about you?