Remove cmd

src/mpsd_software_manager/mpsd_software.py

@@ -1082,7 +1082,7 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove
     """Remove release from installation.
 
     Handle 3 situations :
-    1. remove dosent specify what to remove
+    1. remove does not specify what to remove
         -> warn and exit
     2. remove all package_sets from release
         -> remove release folder except logs
@@ -1097,7 +1097,7 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove
         A Path object representing the path to the directory where
         the release and package_sets will be installed.
     package_sets : list of str
-        A list of strings representing the package_sets to install
+        A list of strings representing the package_sets to remove
         (e.g., "foss2021a-mpi", "global_generic", "ALL").
     force_remove : bool, optional
         A boolean indicating whether to force remove the release.
@@ -1138,9 +1138,6 @@ def remove_environment(mpsd_release, root_dir, package_sets="NONE", force_remove
 
         logging.info(f"> Logging removal of {mpsd_release} at {build_log_path}")
         folders_to_remove = os.listdir(dir_to_remove)
-        # skip logs folder
-        # if "logs" in folders_to_remove:
-        #     folders_to_remove.remove("logs")
         for folder in folders_to_remove:
             # shutil.rmtree(dir_to_remove / folder) #dosent delete file
             run(
@@ -1190,6 +1187,7 @@ def remove_spack_environment(spack_dir, environment_name, build_log_path=None):
     then uninstall all packages exclusive to the environment,
     then deactivate the environment,
     and finally remove the environment.
+
     Parameters
     ----------
     spack_dir : pathlib.Path