💅🤖 Auto-format entire repository with Ruff

Author: webknjaz

build-scripts/manylinux-container-image/manylinux_mapping.py

@@ -21,10 +21,12 @@
 def to_modern_manylinux_tag(legacy_manylinux_tag):
     """Return a modern alias for the tag if it exists."""
     try:
-        return '_'.join((
-            ML_LEGACY_TO_MODERN_MAP[legacy_manylinux_tag],
-            ARCH,
-        ))
+        return '_'.join(
+            (
+                ML_LEGACY_TO_MODERN_MAP[legacy_manylinux_tag],
+                ARCH,
+            ),
+        )
     except KeyError:
         return legacy_manylinux_tag
 

docs/conf.py

@@ -176,15 +176,13 @@
     r'http://localhost:\d+/',  # local URLs
     r'https://codecov\.io/gh/ansible/pylibssh/branch/devel/graph/badge\.svg',
     r'https://github\.com/ansible/pylibssh/actions',  # 404 if no auth
-
     # Too many links to GitHub so they cause "429 Client Error:
     # too many requests for url"
     # Ref: https://github.com/sphinx-doc/sphinx/issues/7388
     r'https://github\.com/ansible/pylibssh/issues',
     r'https://github\.com/ansible/pylibssh/pull',
     r'https://github\.com/ansible/ansible/issues',
     r'https://github\.com/ansible/ansible/pull',
-
     # Requires a more liberal 'Accept: ' HTTP request header:
     # Ref: https://github.com/sphinx-doc/sphinx/issues/7247
     r'https://github\.com/ansible/pylibssh/workflows/[^/]+/badge\.svg',
@@ -199,7 +197,9 @@
 
 # -- Options for towncrier_draft extension -----------------------------------
 
-towncrier_draft_autoversion_mode = 'draft'  # or: 'sphinx-version', 'sphinx-release'
+towncrier_draft_autoversion_mode = (
+    'draft'  # or: 'sphinx-version', 'sphinx-release'
+)
 towncrier_draft_include_empty = True
 towncrier_draft_working_directory = PROJECT_ROOT_DIR
 towncrier_draft_config_path = 'pyproject.toml'  # relative to cwd

packaging/pep517_backend/_backend.py

@@ -69,7 +69,8 @@
     'get_requires_for_build_wheel',
     'prepare_metadata_for_build_wheel',
     *(
-        () if _setuptools_build_editable is None
+        ()
+        if _setuptools_build_editable is None
         else (
             'build_editable',
             'get_requires_for_build_editable',
@@ -96,11 +97,11 @@ def _is_truthy_setting_value(setting_value: str) -> bool:
 
 
 def _get_setting_value(
-        config_settings: 'dict[str, str] | None' = None,
-        config_setting_name: 'str | None' = None,
-        env_var_name: 'str | None' = None,
-        *,
-        default: bool = False,
+    config_settings: 'dict[str, str] | None' = None,
+    config_setting_name: 'str | None' = None,
+    env_var_name: 'str | None' = None,
+    *,
+    default: bool = False,
 ) -> bool:
     user_provided_setting_sources = (
         (config_settings, config_setting_name, (KeyError, TypeError)),
@@ -117,9 +118,9 @@ def _get_setting_value(
 
 
 def _include_cython_line_tracing(
-        config_settings: 'dict[str, str] | None' = None,
-        *,
-        default=False,
+    config_settings: 'dict[str, str] | None' = None,
+    *,
+    default=False,
 ) -> bool:
     return _get_setting_value(
         config_settings,
@@ -202,7 +203,7 @@ def _exclude_dir_path(
     ]
     if visited_directory_subdirs_to_ignore:
         print(  # noqa: WPS421
-            f'Preventing `{excluded_dir_path !s}` from being '
+            f'Preventing `{excluded_dir_path!s}` from being '
             'copied into itself recursively...',
             file=_standard_error_stream,
         )
@@ -230,9 +231,9 @@ def _in_temporary_directory(src_dir: Path) -> t.Iterator[None]:
 
 @contextmanager
 def _prebuild_c_extensions(
-        line_trace_cython_when_unset: bool = False,
-        build_inplace: bool = False,
-        config_settings: 'dict[str, str] | None' = None,
+    line_trace_cython_when_unset: bool = False,
+    build_inplace: bool = False,
+    config_settings: 'dict[str, str] | None' = None,
 ) -> t.Generator[None, t.Any, t.Any]:
     """Pre-build C-extensions in a temporary directory, when needed.
 
@@ -248,7 +249,8 @@ def _prebuild_c_extensions(
     )
 
     build_dir_ctx = (
-        nullcontext_cm() if build_inplace
+        nullcontext_cm()
+        if build_inplace
         else _in_temporary_directory(src_dir=Path.cwd().resolve())
     )
     with build_dir_ctx:
@@ -264,9 +266,9 @@ def _prebuild_c_extensions(
 
 @patched_dist_get_long_description()
 def build_wheel(
-        wheel_directory: str,
-        config_settings: 'dict[str, str] | None' = None,
-        metadata_directory: 'str | None' = None,
+    wheel_directory: str,
+    config_settings: 'dict[str, str] | None' = None,
+    metadata_directory: 'str | None' = None,
 ) -> str:
     """Produce a built wheel.
 
@@ -278,9 +280,9 @@ def build_wheel(
 
     """
     with _prebuild_c_extensions(
-            line_trace_cython_when_unset=False,
-            build_inplace=False,
-            config_settings=config_settings,
+        line_trace_cython_when_unset=False,
+        build_inplace=False,
+        config_settings=config_settings,
     ):
         return _setuptools_build_wheel(
             wheel_directory=wheel_directory,
@@ -291,9 +293,9 @@ def build_wheel(
 
 @patched_dist_get_long_description()
 def build_editable(
-        wheel_directory: str,
-        config_settings: 'dict[str, str] | None' = None,
-        metadata_directory: 'str | None' = None,
+    wheel_directory: str,
+    config_settings: 'dict[str, str] | None' = None,
+    metadata_directory: 'str | None' = None,
 ) -> str:
     """Produce a built wheel for editable installs.
 
@@ -305,9 +307,9 @@ def build_editable(
 
     """
     with _prebuild_c_extensions(
-            line_trace_cython_when_unset=True,
-            build_inplace=True,
-            config_settings=config_settings,
+        line_trace_cython_when_unset=True,
+        build_inplace=True,
+        config_settings=config_settings,
     ):
         return _setuptools_build_editable(
             wheel_directory=wheel_directory,
@@ -317,7 +319,7 @@ def build_editable(
 
 
 def get_requires_for_build_wheel(
-        config_settings: 'dict[str, str] | None' = None,
+    config_settings: 'dict[str, str] | None' = None,
 ) -> 'list[str]':
     """Determine additional requirements for building wheels.
 
@@ -330,9 +332,12 @@ def get_requires_for_build_wheel(
         'Cython; python_version < "3.12"',
     ]
 
-    return _setuptools_get_requires_for_build_wheel(
-        config_settings=config_settings,
-    ) + c_ext_build_deps
+    return (
+        _setuptools_get_requires_for_build_wheel(
+            config_settings=config_settings,
+        )
+        + c_ext_build_deps
+    )
 
 
 build_sdist = patched_dist_get_long_description()(_setuptools_build_sdist)

packaging/pep517_backend/_transformers.py

@@ -76,9 +76,7 @@ def sanitize_rst_roles(rst_source_text: str) -> str:  # noqa: WPS210
     gh_role_regex = r"""(?x)
         :gh:`(?P<gh_slug>[^`<]+)(?:\s+([^`]*))?`
     """
-    gh_substitution_pattern = (
-        r'GitHub: ``\g<gh_slug>``'
-    )
+    gh_substitution_pattern = r'GitHub: ``\g<gh_slug>``'
 
     meth_role_regex = r"""(?x)
         (?::py)?:meth:`~?(?P<rendered_text>[^`<]+)(?:\s+([^`]*))?`

packaging/pep517_backend/hooks.py

@@ -10,14 +10,19 @@
 
 # Re-exporting PEP 517 hooks
 from ._backend import (  # type: ignore[assignment]
-    build_sdist, build_wheel, get_requires_for_build_wheel,
+    build_sdist,
+    build_wheel,
+    get_requires_for_build_wheel,
     prepare_metadata_for_build_wheel,
 )
 
 
-with _suppress(ImportError):  # Only succeeds w/ setuptools implementing PEP 660
+with _suppress(
+    ImportError,
+):  # Only succeeds w/ setuptools implementing PEP 660
     # Re-exporting PEP 660 hooks
     from ._backend import (  # type: ignore[assignment]
-        build_editable, get_requires_for_build_editable,
+        build_editable,
+        get_requires_for_build_editable,
         prepare_metadata_for_build_editable,
     )

src/pylibsshext/__init__.py

@@ -3,5 +3,8 @@
 """Python bindings for libssh."""
 
 from ._version import (  # noqa: F401, WPS300
-    __full_version__, __libssh_version__, __version__, __version_info__,
+    __full_version__,
+    __libssh_version__,
+    __version__,
+    __version_info__,
 )

src/pylibsshext/_version.py

@@ -11,12 +11,15 @@
     from ._scm_version import version as __version__  # noqa: WPS300
 except ImportError:
     from pkg_resources import get_distribution as _get_dist
+
     __version__ = _get_dist('ansible-pylibssh').version
 
 
 __full_version__ = (
-    '<pylibsshext v{wrapper_ver!s} with libssh v{backend_ver!s}>'.
-    format(wrapper_ver=__version__, backend_ver=__libssh_version__)
+    '<pylibsshext v{wrapper_ver!s} with libssh v{backend_ver!s}>'.format(
+        wrapper_ver=__version__,
+        backend_ver=__libssh_version__,
+    )
 )
 __version_info__ = tuple(
     (int(chunk) if chunk.isdigit() else chunk)

tests/_service_utils.py

@@ -52,7 +52,8 @@ def wait_for_svc_ready_state(
         '-oUserKnownHostsFile=/dev/null',
         '-oStrictHostKeyChecking=no',
         host,
-        '--', 'exit 0',
+        '--',
+        'exit 0',
     ]
 
     attempts = 0
@@ -68,8 +69,10 @@ def wait_for_svc_ready_state(
 
 
 def ensure_ssh_session_connected(
-        ssh_session, sshd_addr, ssh_clientkey_path,
-        ssh_session_retries=0,
+    ssh_session,
+    sshd_addr,
+    ssh_clientkey_path,
+    ssh_session_retries=0,
 ):
     """Attempt connecting to the SSH server until successful.
 

tests/conftest.py

@@ -11,7 +11,8 @@
 
 import pytest
 from _service_utils import (
-    ensure_ssh_session_connected, wait_for_svc_ready_state,
+    ensure_ssh_session_connected,
+    wait_for_svc_ready_state,
 )
 
 from pylibsshext.session import Session
@@ -98,7 +99,8 @@ def ssh_clientkey_path(sshd_path):
         '-tecdsa',
         '-b256',
         '-Cansible-pylibssh integration tests key',
-        '-N', '',  # empty string for no password
+        '-N',
+        '',  # empty string for no password
         f'-f{path!s}',
     )
     subprocess.check_call(keygen_cmd)
@@ -179,7 +181,13 @@ def ssh_authorized_keys_path(sshd_path, ssh_clientkey_path):
 
 
 @pytest.fixture
-def sshd_addr(free_port_num, ssh_authorized_keys_path, sshd_hostkey_path, sshd_path, ssh_clientkey_path):
+def sshd_addr(
+    free_port_num,
+    ssh_authorized_keys_path,
+    sshd_hostkey_path,
+    sshd_path,
+    ssh_clientkey_path,
+):
     """Spawn an instance of sshd on a free port.
 
     :raises RuntimeError: If spawning SSHD failed.
@@ -198,14 +206,12 @@ def sshd_addr(free_port_num, ssh_authorized_keys_path, sshd_hostkey_path, sshd_p
         '-oLogLevel=DEBUG3',
         f'-oHostKey={sshd_hostkey_path!s}',
         '-oPidFile={pid!s}'.format(pid=sshd_path / 'sshd.pid'),
-
         # NOTE: 'UsePAM no' is not supported on Fedora.
         # Ref: https://bugzilla.redhat.com/show_bug.cgi?id=770756#c1
         '-oUsePAM=yes',
         '-oPasswordAuthentication=no',
         '-oChallengeResponseAuthentication=no',
         '-oGSSAPIAuthentication=no',
-
         '-oStrictModes=no',
         '-oPermitEmptyPasswords=yes',
         '-oPermitRootLogin=yes',

tests/unit/channel_test.py

@@ -79,15 +79,15 @@ def exec_second_command(ssh_channel):
     u_cmd = ssh_channel.exec_command('echo -n Hello Again')
     assert u_cmd.returncode == 0
     assert u_cmd.stderr.decode() == ''
-    assert u_cmd.stdout.decode() == u'Hello Again'
+    assert u_cmd.stdout.decode() == 'Hello Again'
 
 
 def test_exec_command(ssh_channel):
     """Test getting the output of a remotely executed command."""
     u_cmd = ssh_channel.exec_command('echo -n Hello World')
     assert u_cmd.returncode == 0
     assert u_cmd.stderr.decode() == ''
-    assert u_cmd.stdout.decode() == u'Hello World'
+    assert u_cmd.stdout.decode() == 'Hello World'
     # Test that repeated calls to exec_command do not segfault.
 
     # NOTE: Call `exec_command()` once again from another function to
@@ -102,7 +102,7 @@ def test_exec_command_stderr(ssh_channel):
     """Test getting the stderr of a remotely executed command."""
     u_cmd = ssh_channel.exec_command('echo -n Hello World 1>&2')
     assert u_cmd.returncode == 0
-    assert u_cmd.stderr.decode() == u'Hello World'
+    assert u_cmd.stderr.decode() == 'Hello World'
     assert u_cmd.stdout.decode() == ''
 
 
@@ -159,7 +159,9 @@ def test_send_eof(ssh_channel):
 
 def test_send_signal(ssh_channel):
     """Test send_signal correctly forwards signal to the process."""
-    ssh_channel.request_exec('bash -c \'trap "exit 1" SIGUSR1; echo ready; sleep 5; exit 0\'')
+    ssh_channel.request_exec(
+        'bash -c \'trap "exit 1" SIGUSR1; echo ready; sleep 5; exit 0\'',
+    )
 
     # Wait until the process is ready to receive signal
     output = ''
@@ -203,6 +205,7 @@ def test_destructor(ssh_session_connect):
 
     Test that this event does not cause a segfault in channels destructor.
     """
+
     def _do_not_crash():  # noqa: WPS430  # required to create a garbage-collection scope
         ssh_session = Session()
         ssh_session_connect(ssh_session)