decrease diff

Author: MaxymVlasov

ruff.toml

@@ -1,6 +1,8 @@
 # Assume Python 3.9
 target-version = "py39"
 
+line-length = 79 # To decrease PR diff size
+
 [format]
 quote-style = "single"
 
@@ -19,7 +21,7 @@ ignore = [
 ]
 
 [lint.isort]
-force-single-line = true
+# force-single-line = true # To decrease PR diff size
 lines-after-imports = 2
 
 [lint.flake8-pytest-style]

src/pre_commit_terraform/_cli.py

@@ -4,12 +4,13 @@
 from typing import cast as cast_to
 
 from ._cli_parsing import initialize_argument_parser
-from ._errors import PreCommitTerraformBaseError
-from ._errors import PreCommitTerraformExit
-from ._errors import PreCommitTerraformRuntimeError
+from ._errors import (
+    PreCommitTerraformBaseError,
+    PreCommitTerraformExit,
+    PreCommitTerraformRuntimeError,
+)
 from ._structs import ReturnCode
-from ._types import CLIAppEntryPointCallableType
-from ._types import ReturnCodeType
+from ._types import CLIAppEntryPointCallableType, ReturnCodeType
 
 
 def invoke_cli_app(cli_args: list[str]) -> ReturnCodeType:
@@ -37,7 +38,8 @@ def invoke_cli_app(cli_args: list[str]) -> ReturnCodeType:
         raise
     except PreCommitTerraformRuntimeError as unhandled_exc:
         print(  # noqa: T201 FIXME
-            f'App execution took an unexpected turn: {unhandled_exc !s}. Exiting...',
+            f'App execution took an unexpected turn: {unhandled_exc !s}.'
+            'Exiting...',
             file=sys.stderr,
         )
         return ReturnCode.ERROR

src/pre_commit_terraform/_types.py

@@ -1,10 +1,8 @@
 """Composite types for annotating in-project code."""
 
-from argparse import ArgumentParser
-from argparse import Namespace
+from argparse import ArgumentParser, Namespace
 from collections.abc import Callable
-from typing import Protocol
-from typing import Union
+from typing import Protocol, Union
 
 from ._structs import ReturnCode
 
@@ -19,7 +17,10 @@ class CLISubcommandModuleProtocol(Protocol):
     CLI_SUBCOMMAND_NAME: str
     """This constant contains a CLI."""
 
-    def populate_argument_parser(self, subcommand_parser: ArgumentParser) -> None:
+    def populate_argument_parser(
+        self,
+        subcommand_parser: ArgumentParser,
+    ) -> None:
         """Run a module hook for populating the subcommand parser."""
 
     def invoke_cli_app(self, parsed_cli_args: Namespace) -> ReturnCodeType:

src/pre_commit_terraform/terraform_docs_replace.py

@@ -3,8 +3,7 @@
 import os
 import subprocess  # noqa: S404. We invoke cli tools
 import warnings
-from argparse import ArgumentParser
-from argparse import Namespace
+from argparse import ArgumentParser, Namespace
 from typing import cast as cast_to
 
 from ._structs import ReturnCode

tests/pytest/_cli_test.py

@@ -1,15 +1,16 @@
 """Tests for the high-level CLI entry point."""
 
-from argparse import ArgumentParser
-from argparse import Namespace
+from argparse import ArgumentParser, Namespace
 
 import pytest
 
 from pre_commit_terraform import _cli_parsing as _cli_parsing_mod
 from pre_commit_terraform._cli import invoke_cli_app
-from pre_commit_terraform._errors import PreCommitTerraformBaseError
-from pre_commit_terraform._errors import PreCommitTerraformExit
-from pre_commit_terraform._errors import PreCommitTerraformRuntimeError
+from pre_commit_terraform._errors import (
+    PreCommitTerraformBaseError,
+    PreCommitTerraformExit,
+    PreCommitTerraformRuntimeError,
+)
 from pre_commit_terraform._structs import ReturnCode
 from pre_commit_terraform._types import ReturnCodeType
 
@@ -52,7 +53,9 @@ class CustomCmdStub:
         CLI_SUBCOMMAND_NAME = 'sentinel'
 
         @staticmethod
-        def populate_argument_parser(subcommand_parser: ArgumentParser) -> None:  # noqa: ARG004
+        def populate_argument_parser(
+            subcommand_parser: ArgumentParser,  # noqa: ARG004
+        ) -> None:
             return None
 
         @staticmethod
@@ -81,7 +84,9 @@ class CustomCmdStub:
         CLI_SUBCOMMAND_NAME = 'sentinel'
 
         @staticmethod
-        def populate_argument_parser(subcommand_parser: ArgumentParser) -> None:  # noqa: ARG004
+        def populate_argument_parser(
+            subcommand_parser: ArgumentParser,
+        ) -> None:
             return None
 
         @staticmethod

tests/pytest/terraform_docs_replace_test.py

@@ -1,15 +1,16 @@
 """Tests for the `replace-docs` subcommand."""
 
-from argparse import ArgumentParser
-from argparse import Namespace
+from argparse import ArgumentParser, Namespace
 from subprocess import CalledProcessError  # noqa: S404. We invoke cli tools
 
 import pytest
 import pytest_mock
 
 from pre_commit_terraform._structs import ReturnCode
-from pre_commit_terraform.terraform_docs_replace import invoke_cli_app
-from pre_commit_terraform.terraform_docs_replace import populate_argument_parser
+from pre_commit_terraform.terraform_docs_replace import (
+    invoke_cli_app,
+    populate_argument_parser,
+)
 from pre_commit_terraform.terraform_docs_replace import (
     subprocess as replace_docs_subprocess_mod,
 )
@@ -53,7 +54,8 @@ def test_check_is_deprecated() -> None:
             ),
             [
                 'terraform-docs --sort-by-required md ./ > .//SENTINEL.md',
-                'terraform-docs --sort-by-required md ./thing > ./thing/SENTINEL.md',
+                'terraform-docs --sort-by-required md ./thing '
+                '> ./thing/SENTINEL.md',
             ],
             id='two-sorted-files',
         ),
@@ -84,7 +86,9 @@ def test_control_flow_positive(
 
     assert invoke_cli_app(parsed_cli_args) == ReturnCode.OK
 
-    executed_commands = [cmd for ((cmd,), _shell) in check_call_mock.call_args_list]
+    executed_commands = [
+        cmd for ((cmd,), _shell) in check_call_mock.call_args_list
+    ]
 
     assert len(expected_cmds) == check_call_mock.call_count
     assert expected_cmds == executed_commands