Skip to content

Commit c85ca40

Browse files
authored
[Fix] Add missing dateutil dep (#391)
## Problem SDK version 5.3.0 was accidentally shipped with a missing dependency. This wasn't caught in testing because the dependency was present in our `poetry.lock`, which is used when installing in CI, but not in `pyproject.toml` so it wasn't included for the published package. We got into this state because we used to have a dependency on `python-dateutil` which was removed and later re-added when date parsing was needed for support of import endpoints (added 5.3.0). Additional manual testing in google colab didn't catch this either because that environment loads `python-dateutil` as part of the default environment. 🤦‍♀️ ## Solution Add missing dependency. ## Type of Change - [x] Bug fix (non-breaking change which fixes an issue)
1 parent 9892030 commit c85ca40

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

poetry.lock

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ protobuf = { version = "^4.25", optional = true }
6969
protoc-gen-openapiv2 = {version = "^0.0.1", optional = true }
7070
pinecone-plugin-interface = "^0.0.7"
7171
pinecone-plugin-inference = "^1.1.0"
72+
python-dateutil = ">=2.5.3"
7273

7374
[tool.poetry.group.types]
7475
optional = true

0 commit comments

Comments
 (0)