refactor(core): rename partitionning info in use method and update doc

Rawleenc · Oct 25, 2023 · 6805183 · 6805183
1 parent 1943c45
commit 6805183
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/archcraftsman/installer.py b/archcraftsman/installer.py
@@ -126,13 +126,13 @@ def install():
             )
 
         if (
-            archcraftsman.info.ai.partitioning_info.filesystems_in_use()
+            archcraftsman.info.ai.partitioning_info.filesystem_in_use()
             == archcraftsman.options.FSFormats.BTRFS
         ):
             pkgs.add("btrfs-progs")
 
         if (
-            archcraftsman.info.ai.partitioning_info.filesystems_in_use()
+            archcraftsman.info.ai.partitioning_info.filesystem_in_use()
             == archcraftsman.options.FSFormats.XFS
         ):
             pkgs.add("xfsprogs")

diff --git a/archcraftsman/partitioninginfo.py b/archcraftsman/partitioninginfo.py
@@ -41,13 +41,14 @@ def __init__(self, swapfile_size: str = "", main_disk: str = "") -> None:
         self._btrfs_in_use = False
         self._xfs_in_use = False
 
-    def filesystems_in_use(self) -> archcraftsman.options.FSFormats:
+    def filesystem_in_use(self) -> archcraftsman.options.FSFormats:
         """
-        The filesystems in use retrieving method.
+        A method to retrieve the main filesystem in use in this partitioning.
+        Falls back to ext4 if no particular filesystem is in use.
         """
         if self._btrfs_in_use:
             return archcraftsman.options.FSFormats.BTRFS
-        elif self._xfs_in_use:
+        if self._xfs_in_use:
             return archcraftsman.options.FSFormats.XFS
         return archcraftsman.options.FSFormats.EXT4