Always run flake8 through pre-commit, and with doctests (home-assista…

…nt#28490) * Enable flake8 doctests everywhere * Always run flake8 through pre-commit
iDVB · Nov 6, 2019 · bb37bc3 · bb37bc3
1 parent d9edd42
commit bb37bc3
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/.vscode/tasks.json b/.vscode/tasks.json
@@ -33,7 +33,7 @@
     {
       "label": "Flake8",
       "type": "shell",
-      "command": "flake8 homeassistant tests",
+      "command": "pre-commit run flake8 --all-files",
       "group": {
         "kind": "test",
         "isDefault": true

diff --git a/script/lazytox.py b/script/lazytox.py
@@ -120,7 +120,7 @@ async def pylint(files):
 
 async def flake8(files):
     """Exec flake8."""
-    _, log = await async_exec("flake8", "--doctests", *files)
+    _, log = await async_exec("pre-commit", "run", "flake8", "--files", *files)
     res = []
     for line in log.splitlines():
         line = line.split(":")

diff --git a/script/lint b/script/lint
@@ -15,7 +15,7 @@ printf "%s\n" $files
 echo "================"
 echo "LINT with flake8"
 echo "================"
-flake8 --doctests $files
+pre-commit run flake8 --files $files
 echo "================"
 echo "LINT with pylint"
 echo "================"

diff --git a/setup.cfg b/setup.cfg
@@ -21,6 +21,7 @@ norecursedirs = .git testing_config
 
 [flake8]
 exclude = .venv,.git,.tox,docs,venv,bin,lib,deps,build
+doctests = True
 # To work with Black
 max-line-length = 88
 # E501: line too long