PEP8

converters/base.py

@@ -63,7 +63,7 b' class Converter(object):'
     user_preamble = None
     output = unicode()
     raw_as_verbatim = False
-    
+
     def __init__(self, infile):
         self.infile = infile
         self.infile_dir, infile_root = os.path.split(infile)
@@ -168,7 +168,7 b' class Converter(object):'
 
         Returns a path relative to the input file.
         """
-        figname = '%s_fig_%02i.%s' % (self.infile_root, 
+        figname = '%s_fig_%02i.%s' % (self.infile_root,
                                       self.figures_counter, fmt)
         self.figures_counter += 1
         fullname = os.path.join(self.files_dir, figname)
@@ -179,10 +179,10 b' class Converter(object):'
             fopen = lambda fname: open(fname, 'wb')
         else:
             fopen = lambda fname: codecs.open(fname, 'wb', self.default_encoding)
-            
+
         with fopen(fullname) as f:
             f.write(data)
-            
+
         return fullname
 
     def render_heading(self, cell):

converters/notebook.py

@@ -7,7 +7,7 b' class ConverterNotebook(Converter):'
     """
     A converter that is essentially a null-op.
     This exists so it can be subclassed
-    for custom handlers of .ipynb files 
+    for custom handlers of .ipynb files
     that create new .ipynb files.
 
     What distinguishes this from JSONWriter is that

tests/test_references.py

@@ -1,46 +1,48 b''
 import os
 import io
 import nose.tools as nt
-from nbconvert import *
+from nbconvert import (
+    ConverterLaTeX, ConverterMarkdown, ConverterPy, ConverterHTML
+)
 from nose.tools import nottest
 
 
 def test_evens():
-    ###### 
-    # for now, we don't need to really run inkscape to extract svg 
+    ######
+    # for now, we don't need to really run inkscape to extract svg
     # from file, on unix, for test, we monkeypathc it to 'true'
     # which does not fail as doing anything.
     ####
     ConverterLaTeX.inkscape = 'true'
 
-    # commenting rst for now as travis build 
+    # commenting rst for now as travis build
     # fail because of pandoc version.
     converters = [
                  #(ConverterRST,'rst'),
-                  (ConverterMarkdown,'md'),
-                  (ConverterLaTeX,'tex'),
-                  (ConverterPy,'py'),
-                  (ConverterHTML,'html')
+                  (ConverterMarkdown, 'md'),
+                  (ConverterLaTeX, 'tex'),
+                  (ConverterPy, 'py'),
+                  (ConverterHTML, 'html')
                 ]
     reflist = [
             'tests/ipynbref/IntroNumPy.orig'
             ]
-    for root in reflist :
-        for conv,ext in converters:
-            yield test_conversion, conv,root+'.ipynb',root+'.'+ext
+    for root in reflist:
+        for conv, ext in converters:
+            yield test_conversion, conv, root + '.ipynb', root + '.' + ext
+
 
 @nottest
 def compfiles(stra, strb):
-    nt.assert_equal(map(unicode.strip,stra.split('\n')),map(unicode.strip,strb.split('\n')))
+    nt.assert_equal(map(unicode.strip, stra.split('\n')),
+                    map(unicode.strip, strb.split('\n')))
+
 
 @nottest
 def test_conversion(ConverterClass, ipynb, ref_file):
-
     converter = ConverterClass(ipynb)
     converter.read()
-    cv =converter.convert()
+    cv = converter.convert()
     with io.open(ref_file) as ref:
         value = ref.read()
-        compfiles(cv,value)
-    
-
+        compfiles(cv, value)

tests/test_simple.py

@@ -39,8 +39,8 b' def test_render_heading():'
     for level in xrange(1, 7):
         cell = {
             'cell_type': 'heading',
-            'level'    : level,
-            'source'   :  ['Test for heading type H{0}'.format(level)]
+            'level': level,
+            'source':  ['Test for heading type H{0}'.format(level)]
             }
         # Convert cell dictionaries to NotebookNode
         cell_nb = nbformat.NotebookNode(cell)
@@ -53,7 +53,8 b' def test_render_heading():'
         # Render to rst
         c = ConverterRST('')
         rst_list = c.render_heading(cell_nb)
-        nt.assert_true(isinstance(rst_list, list))  # render should return a list
+        # render should return a list
+        nt.assert_true(isinstance(rst_list, list))
         rst_str = "".join(rst_list)
         # Confirm rst content
         chk_str = "Test for heading type H{0}\n{1}\n".format(