Don't run flake8 twice in CI (#15129)

Author: AlexWaygood

.github/workflows/test.yml

@@ -93,6 +93,9 @@ jobs:
           arch: x64
           os: windows-latest
           toxenv: type
+          # We also run these checks with pre-commit in CI,
+          # but it's useful to run them with tox too,
+          # to ensure the tox env works as expected
         - name: Formatting with Black + isort and code style with flake8
           python: '3.7'
           arch: x64

.pre-commit-config.yaml

@@ -14,3 +14,7 @@ repos:
         additional_dependencies:
           - flake8-bugbear==22.12.6  # must match test-requirements.txt
           - flake8-noqa==1.3.0       # must match test-requirements.txt
+
+ci:
+  # We run flake8 as part of our GitHub Actions suite in CI
+  skip: [flake8]