Skip to content

Commit 7018a08

Browse files
SK-3118: Restore python-dotenv dependency for flowvault
The bundled common/ module (common/utils/_utils.py) imports `dotenv`, so `python-dotenv` is a runtime dependency of the flowvault wheel. It was dropped during the bulk removal on the mistaken belief that only the deleted utils/_batching.py used it — but common is vendored into the wheel and needs it. As a result a clean `pip install skyflow-flowvault-python` fails at `import skyflow` with `ModuleNotFoundError: No module named 'dotenv'`. Re-add `python-dotenv >= 1.1.0, < 2` (matching skyvault, which kept it). Verified: the rebuilt wheel declares the dep, and a clean-venv install imports `skyflow` successfully. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent cf1e840 commit 7018a08

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

‎flowvault/setup.py‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,7 @@ def run(self):
8989
'PyJWT >= 2.12, < 3',
9090
'cryptography >= 44.0.2',
9191
'httpx >= 0.21.2',
92+
'python-dotenv >= 1.1.0, < 2',
9293
# NOTE: 'requests' intentionally omitted -- only used today by v2's Connection
9394
# controller, which isn't part of v3's scope this round.
9495
],

0 commit comments

Comments
 (0)