Merge pull request #197 from pre-commit/py2

no longer skip pyc -- does not support python 2

Author: asottile

pre_commit_mirror_maker/make_repo.py

@@ -10,9 +10,6 @@
 from pre_commit_mirror_maker.languages import LIST_VERSIONS
 
 
-EXCLUDED_EXTENSIONS = ('.pyc',)
-
-
 def format_files(src: str, dest: str, **fmt_vars: str) -> None:
     """Copies all files inside src into dest while formatting the contents
     of the files into the output.
@@ -34,10 +31,8 @@ def format_files(src: str, dest: str, **fmt_vars: str) -> None:
     assert os.path.exists(dest)
     # Only at the root.  Could be made more complicated and recursive later
     for filename in os.listdir(src):
-        if filename.endswith(EXCLUDED_EXTENSIONS):
-            continue
         # Flat directory structure
-        elif not os.path.isfile(os.path.join(src, filename)):
+        if not os.path.isfile(os.path.join(src, filename)):
             continue
         with open(os.path.join(src, filename)) as f:
             output_contents = f.read().format(**fmt_vars)

tests/make_repo_test.py

@@ -32,19 +32,6 @@ def test_format_files(tmpdir):
     assert dest.join('file3.txt').read() == 'foo bar derp'
 
 
-def test_format_files_skips_pyc(tmpdir):
-    src = tmpdir.join('src').ensure_dir()
-    dest = tmpdir.join('dest').ensure_dir()
-
-    src.join('setup.py').write('# Setup.py')
-    src.join('setup.pyc').write("# Setup.pyc, don't copy me!")
-
-    format_files(src, dest)
-
-    assert dest.join('setup.py').exists()
-    assert not dest.join('setup.pyc').exists()
-
-
 def test_skips_directories(tmpdir):
     src = tmpdir.join('src').ensure_dir()
     dest = tmpdir.join('dest').ensure_dir()