Skip to content

Resolve Python Logger warnings #2173

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

Merged
merged 2 commits into from
Jun 30, 2025
Merged

Conversation

emmanuel-ferdman
Copy link
Contributor

PR Summary

This small PR resolves the deprecation warnings of the logger library:

DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead

Signed-off-by: Emmanuel Ferdman <[email protected]>
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. backend Pertains to the Python backend. data layer Pertains to data layers. labels May 14, 2025
@hayescode
Copy link
Contributor

@willydouhard while we're on warnings, can this be fixed in the literalai libarary?

C:\Users\user\repo\.venv\Lib\site-packages\literalai\helper.py:38: DeprecationWarning: datetime.datetime.utcnow() is deprecated and scheduled for removal in a future version. Use timezone-aware objects to represent datetimes in UTC: datetime.datetime.now(datetime.UTC).
  dt = datetime.utcnow()

@emmanuel-ferdman
Copy link
Contributor Author

@hayescode @willydouhard I pushed a fix for the datetime warnings as well: Chainlit/literalai-python#173

@hayescode
Copy link
Contributor

@emmanuel-ferdman I'm going to move these utc_now() helpers from literalai into chainlit. There's no reason for this dependency.

@hayescode hayescode enabled auto-merge June 30, 2025 16:48
@hayescode hayescode added this pull request to the merge queue Jun 30, 2025
Merged via the queue into Chainlit:main with commit 31fe84a Jun 30, 2025
8 checks passed
This was referenced Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Pertains to the Python backend. data layer Pertains to data layers. size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants