-
Notifications
You must be signed in to change notification settings - Fork 2k
List releases #862
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
List releases #862
Conversation
…ssion instructions
Update params
Add only new tool to the readme
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 implements two new repository tools for listing releases and getting the latest release from GitHub repositories. The implementation follows the existing patterns in the codebase for similar repository tools.
- Adds
ListReleases
andGetLatestRelease
functions torepositories.go
- Registers both new tools in the default toolset
- Provides comprehensive test coverage for both successful and error scenarios
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
pkg/github/tools.go | Registers the new ListReleases and GetLatestRelease tools in the default toolset |
pkg/github/repositories.go | Implements the core functionality for both tools with proper error handling and response formatting |
pkg/github/repositories_test.go | Adds comprehensive test coverage for both tools including success and error cases |
README.md | Updates documentation to include the new tools and their parameters |
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.
🚀
Implement ListReleases and GetLatestRelease tools in repositories.go
Register both tools in the repos toolset
Add comprehensive tests for listing and fetching the latest release in repositories_test.go
All tests pass, confirming correct integration and behavior
Continue work started by @aryasoni98 in #469