Addressed PEP 8 issues

pandas-dev · TomAugspurger · Apr 23, 2018 · Mar 30, 2018 · Mar 30, 2018 · Mar 30, 2018
commit 91ef466c26253b06d4a2b0ab55104a36806f0c17
diff --git a/pandas/core/indexes/base.py b/pandas/core/indexes/base.py
@@ -254,7 +254,7 @@ def __new__(cls, data=None, dtype=None, copy=False, name=None,
 
         if name is not None and not is_hashable(name):
 result = object.__new__(MultiIndex) 
 # we've already validated levels and labels, so shortcut here 
 result._set_levels(levels, copy=copy, validate=False) 
 result._set_labels(labels, copy=copy, validate=False) 
 if names is not None: 
     # handles name validation 
     result._set_names(names) 
 result = object.__new__(MultiIndex) 
  
 # we've already validated levels and labels, so shortcut here 
 result._set_levels(levels, copy=copy, validate=False) 
 result._set_labels(labels, copy=copy, validate=False) 
  
 if names is not None: 
     # handles name validation 
     result._set_names(names) 
             raise TypeError(__class__.__name__ +
-                '.name must be a hashable type')
+                            '.name must be a hashable type')
 
         if fastpath:
             return cls._simple_new(data, name)
@@ -1366,10 +1366,10 @@ def set_names(self, names, level=None, inplace=False):
                 else:
                     if self.nlevels == 1:
                         raise TypeError(__class__.__name__ +
-                            '.name must be a hashable type')
+                                        '.name must be a hashable type')
                     else:
                         raise TypeError(self.__class__.__name__ +
-                            '.name must be a hashable type')
+                                        '.name must be a hashable type')
 
         if level is not None and self.nlevels == 1:
             raise ValueError('Level must be None for non-MultiIndex')

diff --git a/pandas/core/indexes/multi.py b/pandas/core/indexes/multi.py
@@ -647,7 +647,7 @@ def _set_names(self, names, level=None, validate=True):
                 pass
             else:
                 raise TypeError(self.__class__.__name__ +
-                    '.name must be a hashable type')
+                                '.name must be a hashable type')
 
         # GH 15110
         # Don't allow a single string for names in a MultiIndex

diff --git a/pandas/tests/indexes/test_base.py b/pandas/tests/indexes/test_base.py
@@ -448,7 +448,7 @@ def test_constructor_nonhashable_name(self):
         assert isinstance(idx, Index)
         renamed = ['1']
         tm.assert_raises_regex(TypeError, message,
-            idx.rename, name=renamed)
+                               idx.rename, name=renamed)
 
     def test_view_with_args(self):
 

diff --git a/pandas/tests/indexes/test_multi.py b/pandas/tests/indexes/test_multi.py
@@ -622,18 +622,16 @@ def test_constructor_nonhashable_names(self):
         names = ((['foo'], ['bar']))
         message = "MultiIndex.name must be a hashable type"
         tm.assert_raises_regex(TypeError, message,
-            MultiIndex, levels=levels,
-            labels=labels,
-            names=names)
+                               MultiIndex, levels=levels,
+                               labels=labels, names=names)
 
         # With .rename()
         mi = MultiIndex(levels=[[1, 2], [u'one', u'two']],
-                    labels=[[0, 0, 1, 1], [0, 1, 0, 1]],
-                    names=('foo', 'bar'))
+                        labels=[[0, 0, 1, 1], [0, 1, 0, 1]],
+                        names=('foo', 'bar'))
         assert isinstance(mi, MultiIndex)
         renamed = [['foor'], ['barr']]
-        tm.assert_raises_regex(TypeError, message,
-            mi.rename, names=renamed)
+        tm.assert_raises_regex(TypeError, message, mi.rename, names=renamed)
 
     @pytest.mark.parametrize('names', [['a', 'b', 'a'], ['1', '1', '2'],
                                        ['1', 'a', '1']])