Skip to content

Commit 28fc74d

Browse files
committed
🐛 Fix(report_generator.py): remove logger from the function definition to avoid creating a logger when there is not a report
1 parent 0986060 commit 28fc74d

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/vuegen/report_generator.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
def get_report(
1212
report_type: str,
13-
logger: logging.Logger = get_logger("report"),
13+
logger: logging.Logger = None,
1414
config_path: str = None,
1515
dir_path: str = None,
1616
streamlit_autorun: bool = False,
@@ -36,7 +36,11 @@ def get_report(
3636
ValueError
3737
If neither 'config_path' nor 'directory' is provided.
3838
"""
39-
# Initialize the config manager object
39+
# Initialize logger only if it's not provided
40+
if logger is None:
41+
logger = get_logger("report")
42+
43+
# Create the config manager object
4044
config_manager = ConfigManager(logger)
4145

4246
if dir_path:

0 commit comments

Comments
 (0)