From fafb180ffeab5dcf3d17c1b32eec78b75c6e3bc9 Mon Sep 17 00:00:00 2001 From: correctmost <134317971+correctmost@users.noreply.github.com> Date: Tue, 3 Jun 2025 13:17:10 +0000 Subject: [PATCH] Remove unused arguments from internal functions (#3571) This makes it easier to run Pylint's unused-argument checks on the codebase. --- archinstall/lib/disk/filesystem.py | 11 ++--------- archinstall/lib/models/device_model.py | 1 - archinstall/lib/models/mirrors.py | 2 +- archinstall/tui/curses_menu.py | 3 +-- 4 files changed, 4 insertions(+), 13 deletions(-) diff --git a/archinstall/lib/disk/filesystem.py b/archinstall/lib/disk/filesystem.py index 3726606a..c5e29d81 100644 --- a/archinstall/lib/disk/filesystem.py +++ b/archinstall/lib/disk/filesystem.py @@ -63,18 +63,12 @@ class FilesystemHandler: for mod in device_mods: if boot_part := mod.get_boot_partition(): debug(f'Formatting boot partition: {boot_part.dev_path}') - self._format_partitions( - [boot_part], - mod.device_path, - ) + self._format_partitions([boot_part]) self.perform_lvm_operations() else: for mod in device_mods: - self._format_partitions( - mod.partitions, - mod.device_path, - ) + self._format_partitions(mod.partitions) for part_mod in mod.partitions: if part_mod.fs_type == FilesystemType.Btrfs and part_mod.is_create_or_modify(): @@ -83,7 +77,6 @@ class FilesystemHandler: def _format_partitions( self, partitions: list[PartitionModification], - device_path: Path, ) -> None: """ Format can be given an overriding path, for instance /dev/null to test diff --git a/archinstall/lib/models/device_model.py b/archinstall/lib/models/device_model.py index 176ab098..804db4ae 100644 --- a/archinstall/lib/models/device_model.py +++ b/archinstall/lib/models/device_model.py @@ -84,7 +84,6 @@ class DiskLayoutConfiguration: cls, disk_config: _DiskLayoutConfigurationSerialization, enc_password: Password | None = None, - disk_encryption: _DiskEncryptionSerialization | None = None, ) -> DiskLayoutConfiguration | None: from archinstall.lib.disk.device_handler import device_handler diff --git a/archinstall/lib/models/mirrors.py b/archinstall/lib/models/mirrors.py index 0cd3e961..e8eeaaf1 100644 --- a/archinstall/lib/models/mirrors.py +++ b/archinstall/lib/models/mirrors.py @@ -103,7 +103,7 @@ class MirrorStatusEntryV3(BaseModel): return value @model_validator(mode='after') - def debug_output(self, validation_info) -> 'MirrorStatusEntryV3': + def debug_output(self) -> 'MirrorStatusEntryV3': self._hostname, *port = urllib.parse.urlparse(self.url).netloc.split(':', 1) self._port = int(port[0]) if port and len(port) >= 1 else None diff --git a/archinstall/tui/curses_menu.py b/archinstall/tui/curses_menu.py index 704a936c..2d1e2cbb 100644 --- a/archinstall/tui/curses_menu.py +++ b/archinstall/tui/curses_menu.py @@ -140,7 +140,7 @@ class AbstractViewport: # adjust the original rows and cols of the entries as # they need to be shrunk by 1 to make space for the frame - entries = self._adjust_entries(entries, dim) + entries = self._adjust_entries(entries) framed_entries = [ top_ve, @@ -242,7 +242,6 @@ class AbstractViewport: def _adjust_entries( self, entries: list[ViewportEntry], - frame_dim: FrameDim, ) -> list[ViewportEntry]: for entry in entries: # top row frame offset