chore(gpg_secretstore): fix lints

famedly · Feb 29, 2024 · fa8bde0 · fa8bde0
1 parent b4c259d
commit fa8bde0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/plugins/module_utils/gpg_utils.py b/plugins/module_utils/gpg_utils.py
@@ -232,7 +232,9 @@ def __get_recipients_from_pass_file(self, password_slug: str) -> List[str]:
         while base_path.as_posix() != "/":
             if os.path.isfile(base_path / self.pass_gpg_id_file):
                 break
-            print(f"debug: no {self.pass_gpg_id_file} file found on {base_path}, traversing up")
+            # This is not the proper way to do it, but the proper way is extremely complicated
+            # because we are in module_utils and not a module
+            #print(f"debug: no {self.pass_gpg_id_file} file found on {base_path}, traversing up")
             base_path = base_path.parent
         else:
             raise FileNotFoundError(

diff --git a/plugins/modules/gpg_secretstore.py b/plugins/modules/gpg_secretstore.py
@@ -179,7 +179,7 @@
     returned: changed
 warning:
     description: Human-readable warnings that accrued during the task
-    type: str[]
+    type: list
     returned: failed or successful but with warnings
 """
 
@@ -495,7 +495,7 @@ def main():
 
     if result["warning"]:
         for warn_msg in result["warning"]:
-          module.warn(warn_msg)
+            module.warn(warn_msg)
 
     result["diff"]["before"] = "\n".join(result["diff"]["before"]) + "\n"
     result["diff"]["after"] = "\n".join(result["diff"]["after"]) + "\n"