-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
docs[minor]: bedrock tool calling/WSO docs #5735
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -0,0 +1,64 @@ | |||
|
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.
Hey team, just a heads up that I've flagged a change in the PR for review. The added code explicitly accesses environment variables using process.env
, so it's important to ensure that this is handled securely and appropriately. Let me know if you have any questions or need further clarification!
@@ -0,0 +1,35 @@ | |||
|
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.
Hey there! 👋 I've reviewed the code changes in this PR and flagged it for your review. The added code explicitly accesses environment variables using process.env
, so it's important to ensure that this is handled securely and appropriately. Let me know if you have any questions or need further clarification!
@@ -383,3 +385,63 @@ test.skip.each([ |
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.
Hey team, I've flagged this PR for your review as it includes changes that access environment variables using process.env
. Please take a look to ensure that the handling of environment variables aligns with our best practices.
@@ -0,0 +1,63 @@ | |||
import { BedrockChat } from "@langchain/community/chat_models/bedrock"; |
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.
Hey there! I've reviewed the code and flagged a specific change related to accessing environment variables using process.env
. This is to ensure the maintainers review and confirm the handling of environment variables in the code. Let me know if you need further assistance!
@@ -0,0 +1,34 @@ | |||
import { BedrockChat } from "@langchain/community/chat_models/bedrock"; |
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.
Hey there! 👋 I've reviewed the code changes and noticed that environment variables are being accessed using process.env
. I've flagged this for your review to ensure that the handling of environment variables aligns with best practices. Let me know if you have any questions!
No description provided.