litellm icon indicating copy to clipboard operation
litellm copied to clipboard

Remove an unnecessary dependency.

Open mikolajz opened this issue 10 months ago • 3 comments

Title

Remove dependency on importlib_metadata (even on Python 3.8 stdlib works).

Relevant issues

The extra dependency was causing an issue due to another library with lots of strict dependencies.

Type

🐛 Bug Fix 🧹 Refactoring

Changes

[REQUIRED] Testing - Attach a screenshot of any new tests passing locally

For importlib_metadata - even in Python 3.8 this worked:

$ python
Python 3.8.20 (default, Sep  7 2024, 18:35:08) 
[GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from litellm._version import version
/home/mikolajz/.virtualenvs/litellm/lib/python3.8/site-packages/pydantic/_internal/_config.py:345: UserWarning: Valid config keys have changed in V2:
* 'fields' has been removed
  warnings.warn(message, UserWarning)
>>> version
'1.59.9'

mikolajz avatar Jan 28 '25 18:01 mikolajz

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 13, 2025 9:17pm

vercel[bot] avatar Jan 28 '25 18:01 vercel[bot]

@ishaan-jaff Sorry for the delay. Now it's done. How about the typing fix - should I create a separate PR for it?

mikolajz avatar Apr 06 '25 22:04 mikolajz

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Apr 22 '25 22:04 CLAassistant