Skip to content

Commit 54f623f

Browse files
committed
log(feat[cli-sync-formatter]): Add SimpleLogFormatter to CLI sync for clean output
why: Ensure CLI sync command provides clean user-facing output like print() what: - Extend SimpleLogFormatter to vcspull.cli.sync logger - Consolidate CLI logger setup with DRY loop pattern - Maintain clean output for all user-facing CLI commands
1 parent de9b07a commit 54f623f

File tree

1 file changed

+15
-16
lines changed

1 file changed

+15
-16
lines changed

src/vcspull/log.py

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -47,23 +47,22 @@ def setup_logger(
4747
vcspull_logger.addHandler(channel)
4848
vcspull_logger.propagate = False
4949

50-
# Setup simple formatter specifically for CLI add modules
50+
# Setup simple formatter specifically for CLI modules
5151
# These modules provide user-facing output that should be clean
52-
add_logger = logging.getLogger("vcspull.cli.add")
53-
if not add_logger.handlers:
54-
add_channel = logging.StreamHandler()
55-
add_channel.setFormatter(SimpleLogFormatter())
56-
add_logger.setLevel(level)
57-
add_logger.addHandler(add_channel)
58-
add_logger.propagate = False
59-
60-
add_fs_logger = logging.getLogger("vcspull.cli.add_from_fs")
61-
if not add_fs_logger.handlers:
62-
add_fs_channel = logging.StreamHandler()
63-
add_fs_channel.setFormatter(SimpleLogFormatter())
64-
add_fs_logger.setLevel(level)
65-
add_fs_logger.addHandler(add_fs_channel)
66-
add_fs_logger.propagate = False
52+
cli_loggers = [
53+
"vcspull.cli.add",
54+
"vcspull.cli.add_from_fs",
55+
"vcspull.cli.sync",
56+
]
57+
58+
for logger_name in cli_loggers:
59+
cli_logger = logging.getLogger(logger_name)
60+
if not cli_logger.handlers:
61+
cli_channel = logging.StreamHandler()
62+
cli_channel.setFormatter(SimpleLogFormatter())
63+
cli_logger.setLevel(level)
64+
cli_logger.addHandler(cli_channel)
65+
cli_logger.propagate = False
6766

6867
# setup styling for repo loggers
6968
repo_logger = logging.getLogger("libvcs")

0 commit comments

Comments
 (0)