temporary fix of moving registry entiry to end (so no comments are eaten) and printing warning

Signed-off-by: vsoch <[email protected]>

Author: vsoch

shpc/main/settings.py

@@ -126,6 +126,9 @@ def add(self, key, value):
         value = list(set(current))
         self._settings[key] = value
         self.change_validate(key, value)
+        logger.warning(
+            "Warning: Check with shpc config edit - ordering of list can change."
+        )
 
     def remove(self, key, value):
         """
@@ -139,6 +142,9 @@ def remove(self, key, value):
         current.pop(current.index(value))
         self._settings[key] = current
         self.change_validate(key, current)
+        logger.warning(
+            "Warning: Check with shpc config edit - ordering of list can change."
+        )
 
     def set(self, key, value):
         """
@@ -206,13 +212,8 @@ def save(self, filename=None):
             logger.exit("A filename is required to save to.")
         yaml = YAML()
 
-        # This requires Python 3.7 support
-        try:
-            with open(filename, "w") as fd:
-                yaml.dump(self._settings, fd, sort_keys=False)
-        except:
-            with open(filename, "w") as fd:
-                yaml.dump(self._settings, fd)
+        with open(filename, "w") as fd:
+            yaml.dump(self._settings, fd)
 
     def __iter__(self):
         for key, value in self.__dict__.items():

shpc/settings.yml

@@ -13,10 +13,6 @@ config_editor: vim
 # set a default container technology (singularity or podman)
 container_tech: singularity
 
-# Registry Recipes (order of paths here is honored for search path))
-registry:
- - $root_dir/registry
-
 # Lmod or Environment Modules settings 
 # The install directory for modules. Defaults to the install directory/modules
 module_base: $root_dir/modules
@@ -58,3 +54,7 @@ environment_file: 99-shpc.sh
 # container features like gpu will modify generated recipes
 container_features:
   gpu:      # one of null, amd, or nvidia
+
+# Registry Recipes (order of paths here is honored for search path))
+registry:
+ - $root_dir/registry