From 1ea7dadf51d86b197eccf9119fa349fcd75ecbb6 Mon Sep 17 00:00:00 2001 From: James Braza Date: Tue, 20 Feb 2024 13:27:27 -0500 Subject: [PATCH 1/4] Added venv to flake8 excluded directories --- setup.cfg | 3 +++ 1 file changed, 3 insertions(+) diff --git a/setup.cfg b/setup.cfg index 2b5b9ac..2d76c1e 100644 --- a/setup.cfg +++ b/setup.cfg @@ -4,5 +4,8 @@ python-tag = py3 [doc8] max-line-length = 99 +[flake8] +extend-exclude = venv + [isort] force_single_line = true From 822154b981b905303041a71a3bd04056fb41e9cc Mon Sep 17 00:00:00 2001 From: James Braza Date: Tue, 20 Feb 2024 13:30:16 -0500 Subject: [PATCH 2/4] Added flake8-requirements to part of flake8 workflow in CI --- .github/workflows/check.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/check.yaml b/.github/workflows/check.yaml index 24dc9f8..7f49c57 100644 --- a/.github/workflows/check.yaml +++ b/.github/workflows/check.yaml @@ -64,5 +64,5 @@ jobs: python-version: '3.x' - name: Run flake8 Linter run: | - pip install flake8 + pip install -e . flake8 flake8 --count --show-source --statistics src test From 14d9e1e1f1ac89e2f62b2e6724fc1ddc12ed150d Mon Sep 17 00:00:00 2001 From: James Braza Date: Tue, 20 Feb 2024 13:31:42 -0500 Subject: [PATCH 3/4] Checking tomli removal would break CI --- setup.py | 1 - 1 file changed, 1 deletion(-) diff --git a/setup.py b/setup.py index 0ab5f33..929d094 100644 --- a/setup.py +++ b/setup.py @@ -27,7 +27,6 @@ def get_abs_path(pathname): install_requires=[ "flake8 >= 4.0.0", "setuptools >= 10.0.0", - "tomli>=1.2.1; python_version < '3.11'", ], setup_requires=["pytest-runner"], tests_require=["mock", "pytest"], From 6536f445f6265e784fd3ec3fd4f5fa7c397377b9 Mon Sep 17 00:00:00 2001 From: James Braza Date: Tue, 20 Feb 2024 13:33:14 -0500 Subject: [PATCH 4/4] Revert "Checking tomli removal would break CI" This reverts commit 14d9e1e1f1ac89e2f62b2e6724fc1ddc12ed150d. --- setup.py | 1 + 1 file changed, 1 insertion(+) diff --git a/setup.py b/setup.py index 929d094..0ab5f33 100644 --- a/setup.py +++ b/setup.py @@ -27,6 +27,7 @@ def get_abs_path(pathname): install_requires=[ "flake8 >= 4.0.0", "setuptools >= 10.0.0", + "tomli>=1.2.1; python_version < '3.11'", ], setup_requires=["pytest-runner"], tests_require=["mock", "pytest"],