style(ruff): fix linter issues

pyproject.toml

@@ -101,7 +101,7 @@ select = [
     "UP",   # pyupgrade
     "YTT"   # flake8-2020
 ]
-ignore = []
+ignore = ["ANN101", "ANN204"]
 line-length = 120
 target-version = "py312"
 preview = true

sshgen/__main__.py

@@ -2,7 +2,7 @@
 from sshgen.cli import app
 
 
-def main():
+def main() -> None:
     app()
 
 

sshgen/cli.py

@@ -1,8 +1,7 @@
 #!/usr/bin/env python
-from typing import Optional
+from typing import Annotated
 
 import typer
-from typing_extensions import Annotated
 
 from sshgen import __app_name__, __version__
 from sshgen.utils.app import AppUtils
@@ -11,9 +10,9 @@
 app = typer.Typer(no_args_is_help=True)
 
 
-@app.command("generate")
+@app.command('generate')
 def generate_hosts_file(hosts_file: Annotated[str, typer.Option('--hosts-file', '-h')] = './hosts.yml',
-                        output: Annotated[str, typer.Option('--output', '-o')] = "./config"):
+                        output: Annotated[str, typer.Option('--output', '-o')] = './config') -> None:
     """
     Command to generate SSH configuration file.
     By default, it uses hosts.yml file placed in your working directory and outputs to file named as "config"
@@ -27,17 +26,18 @@ def generate_hosts_file(hosts_file: Annotated[str, typer.Option('--hosts-file',
     AppUtils(hosts_file, output_file).generate_ssh_config()
 
 
-def _version_callback(value: bool):
+def _version_callback(value: bool) -> None:
     if value:
         typer.echo(f'{__app_name__} v{__version__}')
         raise typer.Exit()
 
 
+# noinspection PyUnusedLocal
 @app.callback()
 def callback(
-        version: Annotated[Optional[bool],
-        typer.Option('-v', '--version', callback=_version_callback, is_eager=True)] = None
-):
+        version: Annotated[bool | None,  # noqa: ARG001
+        typer.Option('-v', '--version', callback=_version_callback, is_eager=True)] = None,
+) -> None:
     """
     sshgen generates SSH configuration file based on an Ansible hosts file.
     """

sshgen/generators/model.py

@@ -6,7 +6,8 @@
 
 
 class MapToHost:
-    def convert(self, parsed_hosts: CommentedMap) -> list[HostModel]:
+    @staticmethod
+    def convert(parsed_hosts: CommentedMap) -> list[HostModel]:
         host_models = []
 
         for host_group, hosts in parsed_hosts.items():
@@ -15,7 +16,7 @@ def convert(self, parsed_hosts: CommentedMap) -> list[HostModel]:
                     host=host,
                     host_group=host_group,
                     ansible_host=host_details['ansible_host'],
-                    ansible_user=host_details['ansible_user']
+                    ansible_user=host_details['ansible_user'],
                 )
 
                 meta = host_details.get('_meta', {})

sshgen/generators/sshconfig.py

@@ -43,12 +43,13 @@ def _open_template(self) -> str:
     def _save_config(self, templates: list[str]) -> None:
         self._create_output_file()
         self.output_file.write_text('\n'.join(templates))
-        log.info("Generated SSH config file was saved to %s", self.output_file)
+        log.info('Generated SSH config file was saved to %s', self.output_file)
 
     def _create_output_file(self) -> None:
         self.output_file.touch(exist_ok=True)
 
-    def _get_fallback_auth(self) -> list[str]:
+    @staticmethod
+    def _get_fallback_auth() -> list[str]:
         parameter = 'IdentitiesOnly yes'
         width = len(parameter) + 4
         return ['IdentityFile ~/.ssh/ssh_key', f'{parameter: >{width}}']

sshgen/logger.py

@@ -7,7 +7,7 @@ def init_logger(module_name: str) -> Logger:
     logger: Logger = logging.getLogger(module_name)
     logger.setLevel(logging.INFO)
 
-    console_formatter = logging.Formatter('[%(asctime)s] [%(levelname)s] - %(message)s', datefmt="%Y-%m-%d %H:%M:%S")
+    console_formatter = logging.Formatter('[%(asctime)s] [%(levelname)s] - %(message)s', datefmt='%Y-%m-%d %H:%M:%S')
     console_handler = logging.StreamHandler()
     console_handler.setFormatter(console_formatter)
 

sshgen/parsers/ansible.py

@@ -7,7 +7,6 @@
 
 
 class AnsibleHostsParser:
-
     def __init__(self, hosts_file_path: Path | None = None):
         self._yaml = YAML()
         self._default_hosts_file = 'examples/hosts.yml'

sshgen/run.py

@@ -1,10 +1,9 @@
 #!/usr/bin/env python
-
 from sshgen.utils.app import AppUtils
 from sshgen.utils.file import FileUtils
 
 
-def run():
+def run() -> None:
     hosts_file = FileUtils.as_file(file_path='../examples/hosts.yml')
     output_file = FileUtils.as_file(file_path='../config')
 

sshgen/utils/file.py

@@ -47,5 +47,5 @@ def _create_file(file_path: Path) -> None:
                 file_path.parent.mkdir(parents=True, exist_ok=True)
                 file_path.touch(exist_ok=True)
             except OSError as e:
-                log.error("Failed to create file or directory: %s, reason: %s", file_path, e.strerror)
+                log.exception('Failed to create file or directory: %s, reason: %s', file_path, e.strerror)
                 sys.exit(1)