Make the longest module name length function more generic

Signed-off-by: jakub-nt <[email protected]>

cfbs/build.py

@@ -231,7 +231,7 @@ def perform_build_steps(config) -> int:
         user_error("No 'build' key found in the configuration")
         return 1
     print("\nSteps:")
-    module_name_length = config.longest_module_name()
+    module_name_length = config.longest_module_key_length("name")
     for module in config.get("build", []):
         for step in module["steps"]:
             _perform_build_step(module, step, module_name_length)

cfbs/cfbs_config.py

@@ -80,8 +80,8 @@ def save(self):
         with open(self.path, "w") as f:
             f.write(data)
 
-    def longest_module_name(self) -> int:
-        return max((len(m["name"]) for m in self["build"])) if self.get("build") else 0
+    def longest_module_key_length(self, key) -> int:
+        return max((len(m[key]) for m in self["build"])) if self.get("build") else 0
 
     def add_with_dependencies(self, module, remote_config=None, dependent=None):
         if type(module) is list:

cfbs/commands.py

@@ -302,7 +302,7 @@ def status_command() -> int:
     if not modules:
         return 0
     print("\nModules:")
-    max_length = config.longest_module_name()
+    max_length = config.longest_module_key_length("name")
     counter = 1
     for m in modules:
         if m["name"].startswith("./"):
@@ -843,7 +843,7 @@ def _download_dependencies(
     #       2. Code for copying things into ./out
     print("\nModules:")
     counter = 1
-    max_length = config.longest_module_name()
+    max_length = config.longest_module_key_length("name")
     downloads = os.path.join(cfbs_dir(), "downloads")
     for module in config.get("build", []):
         name = module["name"]