Ensure that we don't check in code with print statements. (#110)

docs-sphinx/prepare_docstrings.py

@@ -4,6 +4,7 @@
 import inspect
 import pkgutil
 import os.path
+import sys
 
 
 order = [
@@ -41,9 +42,9 @@ def ensure(objname, filename, content):
         overwrite = open(filename, "r").read() != content
     if overwrite:
         open(filename, "w").write(content)
-        print(objname, "(OVERWRITTEN)")
+        sys.err.write(objname + " (OVERWRITTEN)\n")
     else:
-        print(objname)
+        sys.err.write(objname + "\n")
 
 
 def handle_module(modulename, module):

requirements-test.txt

@@ -1,5 +1,6 @@
 pytest
 flake8
+flake8-print
 scikit-hep-testdata
 lz4
 xxhash

setup.cfg

@@ -1,5 +1,5 @@
 [flake8]
-ignore = E203, W503, E501, E266
+ignore = E203, W503, E501, E266, N
 max-complexity = 100
 exclude = tests, setup.py, uproot4/__init__.py
 

uproot4/reading.py

@@ -112,7 +112,6 @@ def open(
     * :py:func:`~uproot4.behaviors.TBranch.lazy`: returns a lazily read array from
       ``TTrees``.
     """
-
     if isinstance(path, dict) and len(path) == 1:
         ((file_path, object_path),) = path.items()
 

uproot4/source/futures.py

@@ -167,7 +167,7 @@ def run(self):
         """
         future = None
         while True:
-            del future   # don't hang onto a reference while waiting for more work
+            del future  # don't hang onto a reference while waiting for more work
             future = self._work_queue.get()
             if future is None:
                 break
@@ -323,7 +323,7 @@ def run(self):
         """
         future = None
         while True:
-            del future   # don't hang onto a reference while waiting for more work
+            del future  # don't hang onto a reference while waiting for more work
             future = self._work_queue.get()
             if future is None:
                 break