Merge pull request #11828 from proinsias/use_is_or_is_not

CLN: Use `is` operator for comparing with `None` (Pep8)

Author: jreback

pandas/core/generic.py

@@ -4499,7 +4499,7 @@ def describe_1d(data, percentiles):
             else:
                 data = self
         elif include == 'all':
-            if exclude != None:
+            if exclude is not None:
                 msg = "exclude must be None when include is 'all'"
                 raise ValueError(msg)
             data = self

pandas/core/groupby.py

@@ -586,7 +586,7 @@ def wrapper(*args, **kwargs):
             # a little trickery for aggregation functions that need an axis
             # argument
             kwargs_with_axis = kwargs.copy()
-            if 'axis' not in kwargs_with_axis or kwargs_with_axis['axis']==None:
+            if 'axis' not in kwargs_with_axis or kwargs_with_axis['axis'] is None:
                 kwargs_with_axis['axis'] = self.axis
 
             def curried_with_axis(x):

pandas/tests/test_base.py

@@ -293,10 +293,12 @@ def test_none_comparison(self):
 
                 o[0] = np.nan
 
+                # noinspection PyComparisonWithNone
                 result = o == None
                 self.assertFalse(result.iat[0])
                 self.assertFalse(result.iat[1])
 
+                # noinspection PyComparisonWithNone
                 result = o != None
                 self.assertTrue(result.iat[0])
                 self.assertTrue(result.iat[1])

pandas/util/decorators.py

@@ -274,7 +274,7 @@ def make_signature(func) :
     """
     from inspect import getargspec
     spec = getargspec(func)
-    if spec.defaults == None :
+    if spec.defaults is None :
         n_wo_defaults = len(spec.args)
         defaults = ('',) * n_wo_defaults
     else :