Skip to content
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.

Sign up for GitHub

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

Jump to bottom

restrict fastapi, lock langsmith higher version #227

Merged
merged 1 commit into from
Nov 15, 2023
Merged

restrict fastapi, lock langsmith higher version #227

merged 1 commit into from
Nov 15, 2023

Conversation

efriis
Copy link
Member

@efriis efriis commented Nov 14, 2023 •
edited
Loading

2 changes:

  • restrict fastapi version to <1, so it doesn't break the cli test release (test.pypi.org has a fastapi==1 version that is unrelated to real fastapi)
  • upgrade langsmith in the lockfile to 0.0.64 (from 0.0.62, which was yanked)

Some unrelated lockfile changes also happened. Happy to try to revert those but a bit unclear how.

@cla-bot cla-bot bot added the cla-signed label Nov 14, 2023
@efriis efriis requested a review from eyurtsev November 14, 2023 23:26
@efriis
Copy link
Member Author

efriis commented Nov 14, 2023

@eyurtsev maybe I have a different poetry version than you? I'm running 1.6.1. Could also just be linux vs mac

@eyurtsev
Copy link
Collaborator

@efriis i'm using 1.6.1 as well

@eyurtsev eyurtsev merged commit e6323af into main Nov 15, 2023
@eyurtsev eyurtsev deleted the erick/restrict-fastapi--lock-langsmith-higher-version- branch November 15, 2023 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eyurtsev eyurtsev eyurtsev approved these changes

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@efriis @eyurtsev