-
Notifications
You must be signed in to change notification settings - Fork 16.1k
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
community: adds support for getting github releases for the configured repository #29318
community: adds support for getting github releases for the configured repository #29318
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 is a somewhat breaking change. Consider users who are running agents that access tools in this toolkit. If you are adding tools, you are giving these agents new capabilities that users may not desire or be aware of. What do you think about gating these behind a flag? e.g., include_release_tools: bool = False
.
We should also document them in this page (can be modified here).
added the flag which is defaulted to False. This can be used when instantiating the toolkit
Let me know if this isn't the preferred place to apply this, I'm not familiar with the convention |
from langchain_community.utilities.github import GitHubAPIWrapper | ||
|
||
|
||
def test_github_toolkit() -> None: |
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.
Thank you for adding this!
Description: adds support for github tool to query github releases on the configure respository
Issue: N/A
Dependencies: N/A
Twitter handle: @MacsDickinson