-
-
Notifications
You must be signed in to change notification settings - Fork 52
Simplify LitLogger implementation #52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simplify LitLogger implementation #52
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Rebuilds the LitLogger library from scratch by removing legacy implementation, adding a thin compatibility wrapper, updating provider modules to use the new default format, and fixing the documentation link.
- Remove all old LitLogger internals (utils, styles, handlers, core modules).
- Introduce a compatibility wrapper in
webscout/Litlogger/__init__.py
. - Update all TTI provider modules to use
LogFormat.DEFAULT
. - Correct the README link to point at
README.md
.
Reviewed Changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
webscout/Provider/TTI/huggingface/sync_huggingface.py | Switched format from MODERN_EMOJI to DEFAULT |
webscout/Provider/TTI/huggingface/async_huggingface.py | Switched format from MODERN_EMOJI to DEFAULT |
webscout/Provider/TTI/Nexra/sync_nexra.py | Switched format from MODERN_EMOJI to DEFAULT |
webscout/Provider/TTI/Nexra/async_nexra.py | Switched format from MODERN_EMOJI to DEFAULT |
webscout/Litlogger/init.py | Added compatibility wrapper for old imports |
README.md | Updated LitLogger path to README.md |
Comments suppressed due to low confidence (1)
webscout/Litlogger/init.py:2
- This relative import assumes a
litlogger
package at the same level, but the directory is namedLitlogger
. Verify the import path and casing to ensure compatibility on case-sensitive filesystems.
from ..litlogger import *
@@ -98,7 +98,7 @@ | |||
* **[GitAPI](webscout/Extra/GitToolkit/gitapi):** Powerful GitHub data extraction toolkit without authentication requirements for public data | |||
* **[SwiftCLI](webscout/swiftcli/Readme.md):** A powerful and elegant CLI framework for beautiful command-line interfaces | |||
* **[LitPrinter](webscout/litprinter/Readme.md):** Styled console output with rich formatting and colors | |||
* **[LitLogger](webscout/litlogger/Readme.md):** Simplified logging with customizable formats and color schemes | |||
* **[LitLogger](webscout/litlogger/README.md):** Simplified logging with customizable formats and color schemes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link points to webscout/litlogger/README.md
but that file doesn’t exist after removal of the old Readme.md
. Please add or rename the documentation file accordingly or update the path.
* **[LitLogger](webscout/litlogger/README.md):** Simplified logging with customizable formats and color schemes | |
* **[LitLogger](webscout/litlogger/Readme.md):** Simplified logging with customizable formats and color schemes |
Copilot uses AI. Check for mistakes.
@@ -37,7 +37,7 @@ | |||
# Initialize our fire logger and agent 🔥 | |||
logger = Logger( | |||
"HuggingFace", | |||
format=LogFormat.MODERN_EMOJI, | |||
format=LogFormat.DEFAULT, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] Since this default format change is repeated across multiple provider modules, consider extracting the default logger configuration into a shared helper or constant to reduce duplication.
Copilot uses AI. Check for mistakes.
Summary
Testing
python -m py_compile $(git ls-files '*.py')
pytest -q
(fails: command not found)