improve logging for masked functions, formatting

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

Author: desultory

src/ugrd/initramfs_dict.py

@@ -275,7 +275,7 @@ def _process_imports(self, import_type: str, import_value: dict) -> None:
                 import_masks = [import_masks] if isinstance(import_masks, str) else import_masks
                 for mask in import_masks:
                     if mask in function_names:
-                        self.logger.warning("[%s] Skipping import of masked function: %s" % (module_name, mask))
+                        self.logger.warning(f"[{c_(module_name, bright=True)}] Skipping import of masked function: {c_(mask, 'yellow')}")
                         function_names.remove(mask)
                         if import_type == "custom_init":
                             self.logger.warning("Skipping custom init function: %s" % mask)

src/ugrd/initramfs_generator.py

@@ -90,7 +90,9 @@ def build(self) -> None:
         self.config_dict.validate()  # Validate the config after the build tasks have been run
 
         if self.validate and not self.validated:
-            raise ValidationError(f"Failed to validate config. Unprocessed values: {', '.join(list(self['_processing'].keys()))}")
+            raise ValidationError(
+                f"Failed to validate config. Unprocessed values: {', '.join(list(self['_processing'].keys()))}"
+            )
 
         self.generate_init()
         self.pack_build()
@@ -153,7 +155,7 @@ def run_hook(self, hook: str, *args, **kwargs) -> list[str]:
         for function in self["imports"].get(hook, []):
             if function.__name__ in self["masks"].get(hook, []):
                 self.logger.warning(
-                    "[%s] Skipping masked function: %s" % (hook, c_(function.__name__, "yellow", bold=True))
+                    f"[{c_(hook, bright=True)}] Skipping masked function: {c_(function.__name__, 'yellow', bold=True)}"
                 )
                 continue