Move mypy check to a pre-commit hook

jazzband · Jan 1, 2021 · 6ed320e · 6ed320e
1 parent c227673
commit 6ed320e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/.github/workflows/qa.yml b/.github/workflows/qa.yml
@@ -16,7 +16,6 @@ jobs:
       matrix:
         toxenv:
           - checkqa
-          - mypy
           - readme
         python-version:
           - "3.x"

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -17,6 +17,13 @@ repos:
       language_version: python3
       additional_dependencies:
       - flake8-pytest-style
+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: v0.790
+    hooks:
+      - id: mypy
+        # Avoid error: Duplicate module named 'setup'
+        # https://github.com/python/mypy/issues/4008
+        exclude: ^tests/test_data/
   - repo: https://github.com/PyCQA/bandit
     rev: 1.7.0
     hooks:

diff --git a/tox.ini b/tox.ini
@@ -3,7 +3,6 @@ envlist =
     # NOTE: keep this in sync with the env list in .github/workflows/ci.yml.
     py{36,37,38,39,py,py3}-pip{20.1,20.2,20.3,previous,latest,master}-coverage
     checkqa
-    mypy
     readme
 skip_missing_interpreters = True
 
@@ -34,11 +33,6 @@ deps = pre-commit
 commands_pre =
 commands = pre-commit run --all-files --show-diff-on-failure
 
-[testenv:mypy]
-deps = mypy
-commands_pre =
-commands = mypy piptools
-
 [testenv:readme]
 deps = twine
 commands_pre =