-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat(deps): bump supported LangChain package versions to latest #45
base: main
Are you sure you want to change the base?
Conversation
langchain = ">=0.2.10,<0.4.0" | ||
langchain-core = ">=0.2.43,<0.4.0" | ||
langchain-community = ">=0.2.5,<0.4.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Diff via
poetry add "langchain>=0.2.10,<0.4.0" "langchain-core>=0.2.43,<0.4.0" "langchain-community>=0.2.5,<0.4.0"
(and poetry add --group dev "langchain-openai>=0.1.25,<0.4.0"
below)
# 6. Restore original dependencies | ||
print_heading "6. Restoring dependencies" | ||
poetry run pip install -r requirements.bak.txt --quiet | ||
rm requirements.bak.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't necessary in the CI -- we could conditionally disable this if we want to save a few seconds.
poetry.lock
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This entire diff comes from https://github.com/lakeraai/chainguard/pull/45/files#r1986414521
This is my first exposure to poetry, so I may have done this incorrectly -- please let me know if that's the case
@@ -0,0 +1,112 @@ | |||
#!/usr/bin/env bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought this would be the simplest way to achieve confidence that the rest of the diff isn't breaking.
langchain = "^0.2.10" | ||
langchain-core = "^0.2.22" | ||
langchain = ">=0.2.10,<0.4.0" | ||
langchain-core = ">=0.2.43,<0.4.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The lower bound was slightly raised here, as I got dependency conflicts when using original version. It's just a patch bump, though, so it shouldn't be breaking.
53cb321
to
5b4b9d4
Compare
Add compatibility test script to verify functionality across different LangChain versions
Add script to run the test suite against the lowest supported versions