Remove flake8 F401 exclusion for __init__.py files

Instead declare the `__all__` variable as stated by flake8's maintainer: https://stackoverflow.com/a/59438802
best-practice-and-impact · Apr 27, 2021 · 9894063 · 9894063
1 parent 95d60b3
commit 9894063
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 2 deletions.
diff --git a/.flake8 b/.flake8
@@ -6,4 +6,3 @@ exclude = venv*,__pycache__,node_modules,bower_components,migrations
 ignore = D203,W503
 max-complexity = 9
 max-line-length = 120
-per-file-ignores = __init__.py:F401
diff --git a/hooks/__init__.py b/hooks/__init__.py
@@ -1,2 +1,3 @@
 from hooks.post_gen_project import remove_folder, set_aqa_framework, set_request_template
 from hooks.pre_gen_project import check_valid_email_address_format
+__all__ = ("check_valid_email_address_format", "remove_folder", "set_aqa_framework", "set_request_template", )
diff --git a/{{ cookiecutter.repo_name }}/.flake8 b/{{ cookiecutter.repo_name }}/.flake8
@@ -6,4 +6,3 @@ exclude = venv*,__pycache__,node_modules,bower_components,migrations
 ignore = D203,W503
 max-complexity = 9
 max-line-length = 120
-per-file-ignores = __init__.py:F401