This issue is geared towards beginners and first-time contributors.
Task Description:
The current logging for both EffortlessConfig and EffortlessDB is inconsistent. Some methods use print, others use logger.log, and others use logger.debug. Additionally, many outputs do not respect the debug configuration setting.
We need to standardize the logging approach across these classes, ensuring that:
All log outputs are managed through the same logging method.
Logs are only displayed when the debug setting in the configuration is set to True.
Missing or unclear debug messages should be added to functions where appropriate, providing meaningful context about the operation.
Expected Behavior:
The EffortlessConfig and EffortlessDB classes should use a single, consistent logging approach.
All logging should occur only if config.debug is set to True.
Functions missing useful debug messages should include additional logging that provides relevant and clear information for debugging.
Notes:
Avoid copying external code. We encourage contributors to explain their own approach. PRs that include in-depth explanations of the reasoning behind the solution will be prioritized for review.
To Discourage AI Solutions:
When submitting a PR, please:
Write a brief explanation of how your solution works, focusing on why you chose this approach.
Discuss potential challenges you encountered and how you solved them.
Add Resolves #2 in the PR to link it to this issue.
If you do not follow these guidelines, your PR will be closed for AI.
This issue is geared towards beginners and first-time contributors.
Task Description:
The current logging for both EffortlessConfig and EffortlessDB is inconsistent. Some methods use
print
, others uselogger.log
, and others uselogger.debug
. Additionally, many outputs do not respect thedebug
configuration setting.We need to standardize the logging approach across these classes, ensuring that:
debug
setting in the configuration is set toTrue
.Expected Behavior:
EffortlessConfig
andEffortlessDB
classes should use a single, consistent logging approach.config.debug
is set toTrue
.Notes:
To Discourage AI Solutions:
When submitting a PR, please:
Resolves #2
in the PR to link it to this issue.If you do not follow these guidelines, your PR will be closed for AI.