Fix some inconsistent-return-statements Pylint warnings (#4503)

Author: correctmost

archinstall/lib/authentication/authentication_handler.py

@@ -53,7 +53,7 @@ def _update_pam_config(
 	def _add_u2f_entry(self, file: Path, entry: str) -> None:
 		if not file.exists():
 			debug(f'File does not exist: {file}')
-			return None
+			return
 
 		content = file.read_text().splitlines()
 

archinstall/lib/installer.py

@@ -402,7 +402,7 @@ def _mount_lvm_vol(self, volume: LvmVolume) -> None:
 
 	def _mount_luks_partition(self, part_mod: PartitionModification, luks_handler: Luks2) -> None:
 		if not luks_handler.mapper_dev:
-			return None
+			return
 
 		if part_mod.fs_type == FilesystemType.BTRFS and part_mod.btrfs_subvols:
 			# Only mount BTRFS subvolumes that have mountpoints specified

archinstall/lib/output.py

@@ -133,7 +133,7 @@ def log(message: str, level: int = logging.DEBUG) -> None:
 		try:
 			import systemd.journal  # type: ignore[import-not-found]
 		except ModuleNotFoundError:
-			return None
+			return
 
 		log_adapter = logging.getLogger('archinstall')
 		log_fmt = logging.Formatter('[%(levelname)s]: %(message)s')

archinstall/tui/ui/components.py

@@ -350,7 +350,7 @@ def _set_cursor(self) -> None:
 
 	def _set_preview(self, item_id: str) -> None:
 		if self._preview_location is None:
-			return None
+			return
 
 		preview_widget = self.query_one('#preview_content', Label)
 		item = self._group.find_by_id(item_id)
@@ -530,7 +530,7 @@ def on_key(self, event: Key) -> None:
 		selection_list = self.query_one(SelectionList)
 
 		if not selection_list.has_focus or event.key != 'enter':
-			return None
+			return
 
 		if len(self._selected_items) < 1:
 			index = selection_list.highlighted
@@ -743,7 +743,7 @@ def on_key(self, event: Key) -> None:
 			if self._is_btn_focus():
 				item = self._group.focus_item
 				if not item:
-					return None
+					return
 				_ = self.dismiss(Result(ResultType.Selection, _item=item))