Skip to content

Commit c0f053f

Browse files
authored
CODEX Merge pull request #24 from Krafman/codex/fix-use-of-f-strings-in-logging
Fix logging format style
2 parents a5f2657 + a42f0b5 commit c0f053f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/llm_controller/config_loader.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,15 +64,15 @@ def _load_all_configs(self):
6464
except yaml.YAMLError as e:
6565
raise ConfigError(f"Error parsing YAML file {config_file.name}: {e}") from e
6666
except FileNotFoundError: # Should ideally not be hit if glob works, but good for robustness
67-
logger.warning("Config file %s not found during load, though glob listed it.", config_file)
67+
logger.warning(f"Config file {config_file} not found during load, though glob listed it.")
6868
except Exception as e: # Catch other unexpected errors during file load
69-
logger.error("An unexpected error occurred while loading %s: %s", config_file.name, e, exc_info=True)
69+
logger.error(f"An unexpected error occurred while loading {config_file.name}: {e}", exc_info=True)
7070
# Depending on policy, might want to raise ConfigError here too
7171

7272
# This warning is less critical if we have specific checks for mandatory files like llm_config.
7373
# If config_dir exists but self.configs is empty, it means no YAML files were found or loaded.
7474
if not self.configs and self.config_dir.exists(): # Check self.config_dir.exists() to ensure warning is relevant
75-
logger.warning("No configuration files (.yml) were found or successfully loaded from %s", self.config_dir)
75+
logger.warning(f"No configuration files (.yml) were found or successfully loaded from {self.config_dir}")
7676

7777

7878
def get_config(self, name: str, default=None):

0 commit comments

Comments
 (0)