Repository: plannigan/columbo (language: python)

Issue:
Correct lint tool paths
The paths passed to the lint tools [in the CI pipeline](https://github.com/wayfair-incubator/columbo/blob/cd55882f62e35fcae21362417899a0b1c91673b8/.github/workflows/main.yml#L13-L93) diverged from what is [done locally](https://github.com/wayfair-incubator/columbo/blob/cd55882f62e35fcae21362417899a0b1c91673b8/docker/run_tests.sh#L41-L53).

Update them to match. Use configuration files select scanned directories when possible so that these are configured in a single place.


Discussion / hints:

Fix patch:
diff --git a/.bandit b/.bandit
index 7cad8d53..8b1406da 100644
--- a/.bandit
+++ b/.bandit
@@ -1,2 +1,2 @@
 [bandit]
-exclude: *venv*,*env*,*scratch*
+exclude: *venv*,*env*,*scratch*,./test
diff --git a/.flake8 b/.flake8
index b3492321..a8b6b5b1 100644
--- a/.flake8
+++ b/.flake8
@@ -4,4 +4,4 @@ max-complexity = 8
 ignore = W503,E501
 builtins = unicode
 tee = True
-exclude = venv,env,.venv,scratch
+extend_exclude = venv,env,.venv,scratch,.mypy_cache
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
index 7ec7bc6c..7f49c218 100644
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@@ -24,7 +24,7 @@ jobs:
           test-requirements: "true"
 
       - name: Run bandit
-        run: bandit --ini .bandit -r columbo
+        run: bandit --ini .bandit -r .
 
   black:
     runs-on: ubuntu-latest
@@ -41,7 +41,7 @@ jobs:
           test-requirements: "true"
 
       - name: Run black
-        run: black --check columbo tests
+        run: black --check .
 
   flake8:
     runs-on: ubuntu-latest
@@ -57,7 +57,7 @@ jobs:
             test-requirements: "true"
 
         - name: Run flake8
-          run: flake8 columbo tests
+          run: flake8
 
   isort:
     runs-on: ubuntu-latest
@@ -74,7 +74,7 @@ jobs:
           test-requirements: "true"
 
       - name: Run isort
-        run: isort --recursive --check-only columbo tests
+        run: isort --check-only .
 
   mypy:
     runs-on: ubuntu-latest
@@ -91,7 +91,7 @@ jobs:
             test-requirements: "true"
 
         - name: Run mypy
-          run: mypy columbo tests
+          run: mypy
 
 
   renovate:
diff --git a/pyproject.toml b/pyproject.toml
index b3a48042..6e4172bc 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -3,6 +3,10 @@ line-length = 88
 target-version = ["py38", "py39", "py310"]
 
 [tool.mypy]
+files = [
+    "columbo",
+    "tests",
+]
 ignore_missing_imports = true
 
 [tool.isort]


Test patch:
diff --git a/.github/actions/verify-wheel/test_module_import.py b/.github/actions/verify-wheel/test_module_import.py
index 0e79ff0e..ee274149 100644
--- a/.github/actions/verify-wheel/test_module_import.py
+++ b/.github/actions/verify-wheel/test_module_import.py
@@ -4,7 +4,9 @@
 try:
     module_name = sys.argv[1]
 except IndexError:
-    print("module_name must be provided as first argument to test_module_import.py script")
+    print(
+        "module_name must be provided as first argument to test_module_import.py script"
+    )
     sys.exit(1)
 
 try:
diff --git a/docker/run_tests.sh b/docker/run_tests.sh
index 895d80ff..edfc3d79 100755
--- a/docker/run_tests.sh
+++ b/docker/run_tests.sh
@@ -35,19 +35,19 @@ while [[ $# -gt 0 ]]; do
 done
 
 # only generate html locally
-pytest tests --cov-report html
+pytest --cov-report html
 
 echo "Running MyPy..."
-mypy columbo tests
+mypy
 
 echo "Running black..."
-black ${BLACK_ACTION} columbo tests docs/examples/
+black ${BLACK_ACTION} .
 
 echo "Running iSort..."
-isort ${ISORT_ACTION} columbo tests docs/examples/
+isort ${ISORT_ACTION} .
 
 echo "Running flake8..."
-flake8 columbo tests
+flake8
 
 echo "Running bandit..."
-bandit --ini .bandit --quiet -r columbo
+bandit --ini .bandit --quiet -r .