title: pep8 minor annoyance
body: <p>In <code>/tmp/spam.py</code>:</p>
  
  <pre><code>n = 69
  
  if n == True:
      print 'potato'
  </code></pre>
  
  <p>pep8 utility complains about this conditional:</p>
  
  <pre><code>wim@SDFA100461C:/tmp$ pep8 spam.py 
  spam.py:3:6: E712 comparison to True should be 'if cond is True:' or 'if cond:'
  </code></pre>
  
  <ul>
  <li>the first suggestion is wrong/"worse" according to <a href="http://www.python.org/dev/peps/pep-0008/" rel="noreferrer">pep8</a> itself</li>
  <li>the second suggestion changes the behaviour of the code</li>
  </ul>
  
  <p>What is the best practice for a case where you actually <em>do</em> want to check equality with <code>True</code>?  Is identity checking with <code>True</code> using <code>is</code> OK?  Why does pep8 utility offer an alternative which is explicitly discouraged in pep8 itself?</p>