From 11b9f2454ebd0b8f81f272dc1f05b678d17ccd7a Mon Sep 17 00:00:00 2001
From: Lukas Wurzinger <lukas@wrz.one>
Date: Fri, 2 May 2025 00:39:56 +0200
Subject: [PATCH] improve logging

---
 src/forgesync/cli.py    | 2 +-
 src/forgesync/mirror.py | 9 ++++++---
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/forgesync/cli.py b/src/forgesync/cli.py
index 57750f6..fffece2 100644
--- a/src/forgesync/cli.py
+++ b/src/forgesync/cli.py
@@ -29,7 +29,7 @@ class ArgumentParser(Tap):
     "whether mirrors should be recreated"
     mirror_interval: str = "8h0m0s"
     "repository mirror interval"
-    log: str = "WARNING"
+    log: str = "INFO"
     "log level"
     filter: str | None = None
     "filter repositories by this regular expression"
diff --git a/src/forgesync/mirror.py b/src/forgesync/mirror.py
index 2428ab3..8001b04 100644
--- a/src/forgesync/mirror.py
+++ b/src/forgesync/mirror.py
@@ -83,13 +83,14 @@ class PushMirrorer:
         existing_push_mirrors: list[PushMirror],
         config: PushMirrorConfig,
     ) -> PushMirror | None:
-        if not repo.needs_mirror:
-            return None
-
         self.logger.info(
             f"Setting up mirrors for {repo.orig_owner}/{repo.name} to {repo.new_owner}/{repo.name} at {repo.clone_url}"
         )
 
+        if not repo.needs_mirror:
+            self.logger.info("Skipping mirrors")
+            return None
+
         final_config = self.config.overlay(config)
 
         if not final_config.is_valid():
@@ -138,6 +139,8 @@ class PushMirrorer:
                     repo=repo.name,
                 )
 
+        self.logger.info("Finished mirrors")
+
         return new_push_mirror
 
     def mirror_repos(