chore: rename the "mypy" workflow to "tox" and enable pep8

JohnVillalovos · JohnVillalovos · commit 89fc26f11c57 · 2021-04-18T08:13:55.000-07:00
Rename the "mypy" workflow to "tox" Add "pep8" to the list of tox tests to run in addition to "mypy". Also update list of ignores for flake8. Stop ignoring H501 and H803. H501: No errors were reported H803: Has been removed as a check from 'hacking' https://github.com/openstack/hacking/blob/master/setup.cfg
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -12,14 +12,6 @@ env:
   PY_COLORS: 1
 
 jobs:
-  black:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-python@v2
-      - uses: psf/black@stable
-        with:
-          black_args: ". --check"
   commitlint:
     runs-on: ubuntu-latest
     steps:
@@ -28,10 +20,15 @@ jobs:
           fetch-depth: 0
       - uses: wagoid/commitlint-github-action@v3
 
-  mypy:
+  linters:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - uses: actions/setup-python@v2
       - run: pip install --upgrade tox
-      - run: tox -e mypy
+      - name: Run black code formatter (https://black.readthedocs.io/en/stable/)
+        run: tox -e black -- --check
+      - name: Run flake8 (https://flake8.pycqa.org/en/latest/)
+        run: tox -e pep8
+      - name: Run mypy static typing checker (http://mypy-lang.org/)
+        run: tox -e mypy
diff --git a/tox.ini b/tox.ini
@@ -52,7 +52,11 @@ commands = {posargs}
 [flake8]
 exclude = .git,.venv,.tox,dist,doc,*egg,build,
 max-line-length = 88
-ignore = E501,H501,H803,W503
+# We ignore the following because we use black to handle code-formatting
+# E203: Whitespace before ':'
+# E501: Line too long
+# W503: Line break occurred before a binary operator
+ignore = E203,E501,W503
 
 [testenv:docs]
 deps = -r{toxinidir}/rtd-requirements.txt
