feat: Use logger instead of print

Author: EnriqueSoria

annotate_pr_with_ruff/github_utils.py

@@ -6,6 +6,10 @@
 from .ruff import RuffError
 from .ruff import run_cli
 
+import logging
+
+logger = logging.getLogger(__name__)
+
 
 def get_diff(owner: str, repo: str, pr_number: int) -> str:
     return run_cli(
@@ -33,7 +37,13 @@ def get_last_commit(owner: str, repo: str, pr_number: int) -> str:
     ).strip()
 
 
-def submit_review(owner: str, repo: str, pr_number: int, review_message:str, errors: Sequence[RuffError]):
+def submit_review(
+    owner: str,
+    repo: str,
+    pr_number: int,
+    review_message: str,
+    errors: Sequence[RuffError],
+):
     url = f"https://api.github.com/repos/{owner}/{repo}/pulls/{pr_number}/reviews"
 
     body = dict(
@@ -58,5 +68,5 @@ def submit_review(owner: str, repo: str, pr_number: int, review_message:str, err
         headers=headers,
         json=body,
     )
-    print(f">>> post({url}, headers={headers}, body={body})")
-    print(response.status_code, response.json())
+    logger.debug(f">>> post({url}, headers={headers}, body={body})")
+    logger.debug(response.status_code, response.json())

annotate_pr_with_ruff/ruff.py

@@ -5,6 +5,10 @@
 from pathlib import Path
 from typing import Tuple
 
+import logging
+
+logger = logging.getLogger(__name__)
+
 
 @dataclass
 class RuffError:
@@ -29,13 +33,17 @@ def run_cli(*command_args) -> str:
     process = subprocess.Popen(command_args, stdout=subprocess.PIPE)
     stdout, _ = process.communicate()
     output = stdout.decode()
-    print(f""">>> {' '.join(command_args)}""")
-    print(output)
+    logger.debug(f""">>> {' '.join(command_args)}""")
+    logger.debug(output)
     return output
 
 
 def ruff(*files: Tuple[str, ...]):
     if not files:
         files = tuple(str(x) for x in Path(".").iterdir() if x.is_dir())
     output = run_cli("ruff", *files)
-    return [RuffError.from_message_error(error) for error in output.splitlines() if error.startswith(files)]
+    return [
+        RuffError.from_message_error(error)
+        for error in output.splitlines()
+        if error.startswith(files)
+    ]