-
Notifications
You must be signed in to change notification settings - Fork 505
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
feat: Add list_branches tool to view repository branches #151
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.
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
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 so much for opening this PR! 🚀
A couple of things if you don't mind:
- Could you please update the README with the new supported tool
- Re: pagination, please take a look at pkg/github: fix use of per page parameter #137 which we have just merged and changed a bit how we handle pagination. If you could adopt that way of handling pagination in this PR, it'd be really great.
Thank you!
Thank you for the review! I've addressed both points:
All tests are passing and the code is now consistent with the project's pagination handling standards. Let me know if you'd like me to make any additional adjustments! |
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.
I apologize for the back and forth! But would you capitalizing the new tool to be consistent with the main
?
Thanks!
Closes: #141