move slotscheck from pre-commit to tox

diff --git a/.github/workflows/run-on-pr.yaml b/.github/workflows/run-on-pr.yaml
index 94db88ba0a275acbcb311d4f6d919ba9ed5acee7..45f62d772c654cb2a832c41df6cdc79a01a82519 100644 (file)
--- a/.github/workflows/run-on-pr.yaml
+++ b/.github/workflows/run-on-pr.yaml
@@ -84,8 +84,8 @@ jobs:
       - name: Run tests
         run: tox -e mypy ${{ matrix.pytest-args }}
 
-  run-pep8:
-    name: pep8-${{ matrix.python-version }}
+  run-lint:
+    name: lint-${{ matrix.python-version }}
     runs-on: ${{ matrix.os }}
     strategy:
       # run this job using this matrix, excluding some combinations below.
@@ -115,7 +115,7 @@ jobs:
           pip list
 
       - name: Run tests
-        run: tox -e pep8
+        run: tox -e lint
 
   # Arm emulation is quite slow (~20min) so for now just run it when merging to main
   # run-test-arm64:

diff --git a/.github/workflows/run-test.yaml b/.github/workflows/run-test.yaml
index 196e3c1b15ae9ac18ad74ef47751b53b8f4a3fb4..677721d7a6152c11e2f098eac622a710b38f01de 100644 (file)
--- a/.github/workflows/run-test.yaml
+++ b/.github/workflows/run-test.yaml
@@ -156,8 +156,8 @@ jobs:
       - name: Run tests
         run: tox -e mypy ${{ matrix.pytest-args }}
 
-  run-pep8:
-    name: pep8-${{ matrix.python-version }}
+  run-lint:
+    name: lint-${{ matrix.python-version }}
     runs-on: ${{ matrix.os }}
     strategy:
       # run this job using this matrix, excluding some combinations below.
@@ -187,7 +187,7 @@ jobs:
           pip list
 
       - name: Run tests
-        run: tox -e pep8
+        run: tox -e lint
 
   run-pep484:
     name: pep484-${{ matrix.python-version }}

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index d80f8bfc149a4387bb43a5754c9881f82e66d80c..015d80ecfc4d1052fbd708f1f6f3347a38befc1a 100644 (file)
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -27,13 +27,6 @@ repos:
           - pydocstyle
           - pygments
 
--   repo: https://github.com/ariebovenberg/slotscheck
-    rev: v0.11.0
-    hooks:
-    -   id: slotscheck
-        exclude: "^(?!lib/sqlalchemy)"
-        additional_dependencies:
-          - typing_extensions
-          - mypy
-          - greenlet
-        entry: env PYTHONPATH=lib slotscheck -v
+
+
+

diff --git a/tox.ini b/tox.ini
index 3e0c3496f197e441631f0b414cd6e37650294f6d..dcdcd63a66de1169fd1c5e88c6b0ac12b0406bd9 100644 (file)
--- a/tox.ini
+++ b/tox.ini
@@ -151,7 +151,7 @@ commands =
     pytest -m mypy {posargs}
 
 # thanks to https://julien.danjou.info/the-best-flake8-extensions/
-[testenv:pep8]
+[testenv:lint]
 basepython = python3
 deps=
       flake8
@@ -164,9 +164,12 @@ deps=
       pydocstyle
       pygments
       black==21.12b0
+      mypy
+      slotscheck>=0.12,<0.13
 commands =
      flake8 ./lib/ ./test/ ./examples/ setup.py doc/build/conf.py {posargs}
      black --check ./lib/ ./test/ ./examples/ setup.py doc/build/conf.py
+     slotscheck -m sqlalchemy
 
 
 # command run in the github action when cext are active.