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.

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

multiple[patch]: upgrade uuid version #5969

Merged

Conversation

chentschel
Copy link
Contributor

Upgrades current uuid version to support v6 generation.

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 3, 2024
Copy link

vercel bot commented Jul 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 4, 2024 6:42pm
langchainjs-docs ✅ Ready (Inspect) Visit Preview Jul 4, 2024 6:42pm

@dosubot dosubot bot added the auto:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs label Jul 3, 2024
@@ -45,7 +45,7 @@
"process": "^0.11.10",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! I noticed the change in the "uuid" package version from "^9.0.0" to "^10.0.0" in the dependencies section. This might affect the peer dependencies, so I'm flagging this for your review. Keep up the great work!

@@ -50,7 +50,7 @@
"mustache": "^4.2.0",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! I noticed the change in the "uuid" package version from "^9.0.0" to "^10.0.0" in the langchain-core/package.json file. This change doesn't affect the peer/dev/hard dependencies, but I've flagged it for your review. Keep up the great work!

@@ -40,7 +40,7 @@
"dependencies": {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! I noticed a change in the "uuid" dependency from a major version update (^9.0.1 to ^10.0.0) in the package.json file. This might impact the project's dependencies, so I'm flagging this for the maintainers to review.

@@ -37,7 +37,7 @@
"dependencies": {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! I noticed that the dependency changes in this PR have updated the "@langchain/core" and "uuid" dependencies to specific versions, which makes them hard dependencies. I've flagged this for your review. Keep up the great work!

@@ -41,7 +41,7 @@
"license": "MIT",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! 👋 I noticed that the package.json file has a dev dependency change for the "uuid" package from "^9.0.0" to "^10.0.0". This comment is to flag the dev dependency change for the maintainers to review. Keep up the great work!

@@ -37,7 +37,7 @@
"dependencies": {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! I noticed that the package.json file has a dependency change for the "uuid" package, allowing for a wider range of versions. This is flagged for your review as it may impact the project's dependencies. Keep up the great work!

@@ -42,7 +42,7 @@
"@langchain/core": ">0.2.0 <0.3.0",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! 👋 I noticed the package.json change includes an update to the "uuid" dependency version. This change is flagged for maintainers to review, as it may impact peer/dev/hard dependencies. Keep up the great work! 🚀

@@ -41,7 +41,7 @@
"license": "MIT",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey there! I noticed the change in the "uuid" dependency version range in the package.json file. This might impact the peer/dev/hard dependencies, so I'm flagging this for the maintainers to review. Great work on the PR!

@jacoblee93
Copy link
Collaborator

Thank you!

@jacoblee93 jacoblee93 changed the title all[chore]: upgrade uuid version multiple[chore]: upgrade uuid version Jul 4, 2024
@jacoblee93 jacoblee93 changed the title multiple[chore]: upgrade uuid version multiple[patch]: upgrade uuid version Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants