-
Notifications
You must be signed in to change notification settings - Fork 2k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: bump momento dependency and refactor search hit usage #3203
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -26,7 +26,7 @@ | |
"@elastic/elasticsearch": "^8.4.0", | ||
"@getmetal/metal-sdk": "^4.0.0", | ||
"@getzep/zep-js": "^0.9.0", | ||
"@gomomento/sdk": "^1.44.1", | ||
"@gomomento/sdk": "^1.47.1", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bumps the examples Momento client. |
||
"@opensearch-project/opensearch": "^2.2.0", | ||
"@pinecone-database/pinecone": "^1.1.0", | ||
"@planetscale/database": "^1.8.0", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -838,8 +838,8 @@ | |
"@faker-js/faker": "^7.6.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Great work on the PR! I noticed that the dependency on "@gomomento/sdk" and "@gomomento/sdk-core" has been updated to a newer version. This comment is just to flag the change for maintainers to review, as it affects the peer/dev/hard dependencies of the project. |
||
"@getmetal/metal-sdk": "^4.0.0", | ||
"@getzep/zep-js": "^0.9.0", | ||
"@gomomento/sdk": "^1.44.1", | ||
"@gomomento/sdk-core": "^1.44.1", | ||
"@gomomento/sdk": "^1.47.1", | ||
"@gomomento/sdk-core": "^1.47.1", | ||
Comment on lines
-841
to
+842
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bumps the dev dependencies for |
||
"@google-ai/generativelanguage": "^0.2.1", | ||
"@google-cloud/storage": "^6.10.1", | ||
"@huggingface/inference": "^2.6.4", | ||
|
@@ -974,9 +974,9 @@ | |
"@elastic/elasticsearch": "^8.4.0", | ||
"@getmetal/metal-sdk": "*", | ||
"@getzep/zep-js": "^0.9.0", | ||
"@gomomento/sdk": "^1.44.1", | ||
"@gomomento/sdk-core": "^1.44.1", | ||
"@gomomento/sdk-web": "^1.44.1", | ||
"@gomomento/sdk": "^1.47.1", | ||
"@gomomento/sdk-core": "^1.47.1", | ||
"@gomomento/sdk-web": "^1.47.1", | ||
Comment on lines
-977
to
+979
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For the peer dependencies we need |
||
"@google-ai/generativelanguage": "^0.2.1", | ||
"@google-cloud/storage": "^6.10.1", | ||
"@huggingface/inference": "^2.6.4", | ||
|
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.
Great job on the PR! I noticed that the dependency "@gomomento/sdk" has been updated to version "^1.47.1", which is a change in the peer/dev/hard dependencies. This comment is to flag the change for maintainers to review.