- Notifications
You must be signed in to change notification settings - Fork899
add support for the update_issue tool#32
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
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.
Pull Request Overview
This PR adds support for an update_issue tool for modifying issues in a GitHub repository.
- Introduces the updateIssue function in pkg/github/issues.go.
- Adds test coverage for the update_issue tool in pkg/github/issues_test.go.
- Updates README.md with relevant tool documentation.
- Registers the update_issue tool in pkg/github/server.go.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
pkg/github/issues.go | Adds the updateIssue tool with parameters for issue updates. |
pkg/github/issues_test.go | Provides tests to validate the updateIssue tool behavior. |
README.md | Documents the new update_issue tool and its parameters. |
pkg/github/server.go | Registers the update_issue tool (and unintentionally duplicates createIssue). |
Comments suppressed due to low confidence (1)
pkg/github/server.go:43
- Duplicate registration of the createIssue tool is observed which may lead to unintended behavior. Removing the redundant call should resolve this issue.
s.AddTool(createIssue(client, t))
Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon.Learn more
Uh oh!
There was an error while loading.Please reload this page.
8e033fd
to5f5a0a5
Comparefaa2366
intomainUh oh!
There was an error while loading.Please reload this page.
Context
This PR adds support for the
update_issue
tool.