fix: add missing dependency sniffio#59
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds sniffio as an explicit dependency to fix issue #53. The library is imported and used in src/zai/core/_utils/_utils.py for detecting the current async library, but was previously only available as a transitive dependency through httpx → anyio → sniffio.
Changes:
- Added
sniffio = "^1.3.1"to the main dependencies inpyproject.toml - Updated
poetry.lockwith new content hash reflecting the dependency change
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| pyproject.toml | Added sniffio as an explicit dependency to ensure it's available when imported directly |
| poetry.lock | Updated lock file with new content hash; Poetry version also updated from 2.1.4 to 2.2.1 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
You should re-run |
hi, suggest keeping it consistent with others. and avoid cases where multiple version ranges have no intersection. |
Description
Added missing dependency sniffio.
Fixes #53