remove flake8 from githook (#109)

* remove flake8 from githook

* Update CONTRIBUTING.md

* Update dev_setup.sh

.devtools/githooks/pre-commit

@@ -7,9 +7,6 @@ import textwrap
 from pathlib import Path
 from subprocess import CalledProcessError, check_output
 
-# Third-party imports
-import flake8.main.git
-
 
 def type_check() -> int:
     mypy_opts = ['--follow-imports=silent', '--ignore-missing-imports']
@@ -91,12 +88,5 @@ def style_check() -> int:
         return e.returncode
 
 
-def flake8_check() -> int:
-    return flake8.main.git.hook(
-        strict=flake8.main.git.config_for('strict'),
-        lazy=flake8.main.git.config_for('lazy'),
-    )
-
-
 if __name__ == '__main__':
-    sys.exit(type_check() | style_check() | flake8_check())
+    sys.exit(type_check() | style_check())

CONTRIBUTING.md

@@ -44,7 +44,7 @@ GitHub provides additional document on [forking a repository](https://help.githu
 
 For development you need the requirements listed in all `requirements...txt` files.
 
-We use flake8, mypy, and [black](https://github.com/python/black) for style and
+We use [black](https://github.com/python/black) and mypy for style and
 type checking. Any code changes that you contribute should pass these checks.
 
 The easiest way to get set up for development is to run the following script:

dev_setup.sh

@@ -54,8 +54,6 @@ fi
 
 # update location of Git hooks from default (.git/hooks) to the versioned folder .devtools/githooks
 git config core.hooksPath ".devtools/githooks"
-# use the flake8 Git pre-commit hook in strict mode (i.e., fail the commit on error)
-git config --bool flake8.strict true
 
 # install project requirements
 python -m pip install -r requirements-setup.txt