Apply review suggestions

Author: MaxymVlasov

.flake8

@@ -53,9 +53,6 @@ extend-ignore =
   # WPS300: "Forbid imports relative to the current folder" -- we use relative imports
   WPS300,
 
-# WPS410: "Forbid some module-level variables" -- __all__ is a legitimate use case
-allowed-module-metadata=__all__
-
 # https://wemake-python-styleguide.readthedocs.io/en/latest/pages/usage/formatter.html
 #format = wemake
 
@@ -106,3 +103,6 @@ pytest-mark-no-parentheses = true
 
 # wemake-python-styleguide
 show-source = true
+
+# WPS410: "Forbid some module-level variables" -- __all__ is a legitimate use case
+allowed-module-metadata=__all__

src/pre_commit_terraform/_types.py

@@ -15,6 +15,7 @@ class CLISubcommandModuleProtocol(Protocol):
     """A protocol for the subcommand-implementing module shape."""
 
     # WPS115: "Require snake_case for naming class attributes".
+    # This protocol describes module shapes and not regular classes.
     # It's a valid use case as then it's used as constants:
     # "CLI_SUBCOMMAND_NAME: Final[str] = 'hook-name'"" on top level
     CLI_SUBCOMMAND_NAME: str  # noqa: WPS115