Merge pull request #132 from mila-iqia/exclude_virtual_env_from_flake8

flake8 now does not run in .venv
mila-iqia · Oct 8, 2024 · d9ed66b · d9ed66b
2 parents a0c22c0 + 518ffda
commit d9ed66b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.flake8 b/.flake8
@@ -1,4 +1,5 @@
 [flake8]
 max-line-length=100
 ignore=W503,D104,D100,D401
-docstring-convention=google
+docstring-convention=google
+exclude= .venv
diff --git a/hooks/pre-commit b/hooks/pre-commit
@@ -5,7 +5,7 @@ set -e
 
 # linting
 flake8 --ignore D . # Check everything but docstrings
-flake8 --select D --exclude tests/  # Check only the docstrings
+flake8 --select D . --exclude tests/,.venv # Check only the docstrings
 isort --check . # Check imports
 
 # Raise error if any staged notebooks contain outputs