if x:, vs if x == True, vs if x is True

x = 'False'
x = 123

Are both True

Other truth values.

The document explains other values.

As far as the PEP8 reason, its far more semantic to read if this_file_is_green

Other falsey values include 0, '', []. You should just use the if x: version.

I'd like to add a short example where those 3 tests differ:

def test(x):
    print(x, ":", bool(x), x == True, x is True)

test("something")
test(1)
test(True)

The output (pretty formatted):

# "something" : True False False
#           1 : True True  False
#        True : True True  True

The following values in Python are false in the context of if and other logical contexts:

• False
• None
• numeric values equal to 0, such as 0, 0.0, -0.0
• empty strings: '' and u''
• empty containers (such as lists, tuples and dictionaries)
• anything that implements __bool__ (in Python3) to return False, or __nonzero__ (in Python2) to return False or 0.
• anything that doesn't implement __bool__ (in Python3) or __nonzero__ (in Python2), but does implement __len__ to return a value equal to 0

An object is considered "false" if any of those applies, and "true" otherwise, regardless of whether it's actually equal to or identical with False or True

Now, if you've arranged that x is necessarily one of the objects True or False, then you can safely write if x. If you've arranged that the "trueness" of x indicates whether or not to perform the operation, regardless of type, then you can safely write if x. Where you can write that you should prefer to do so, since it's cleaner to read.

Normally, if it is allowed for x to take the value True then you're in one of those two cases, and so you would not write if x is True. The important thing is to correctly document the meaning of x, so that it reflects the test used in the code.

Python programmers are expected to know what's considered true, so if you just document, "runs the function if x is true", then that expresses what your original code does. Documenting it, "runs the function if x is True" would have a different meaning, and is less commonly used precisely because of the style rule in PEP8 that says to test for trueness rather than the specific value True.

However, if you wanted the code to behave differently in the case where x is an empty container from the case where it is None, then you would write something like if x is not None.