-
-
Notifications
You must be signed in to change notification settings - Fork 2
Open
Labels
beginnergood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededpython
Description
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
debugsetting in the configuration is set toTrue. - Missing or unclear debug messages should be added to functions where appropriate, providing meaningful context about the operation.
Expected Behavior:
- The
EffortlessConfigandEffortlessDBclasses should use a single, consistent logging approach. - All logging should occur only if
config.debugis set toTrue. - 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 #2in the PR to link it to this issue.
If you do not follow these guidelines, your PR will be closed for AI.
Hemant-Shashikant-Yadav and FiggyFoxFunc
Metadata
Metadata
Assignees
Labels
beginnergood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededpython