Add ability to view branches for a repo #141#205
Conversation
|
Hi @juruen, |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
This looks great thank you, I just merged a PR that will require some small updates to this one: #195 |
Thanks for the heads-up! Just to clarify — could you please specify what small changes are needed following the merge of #195? |
aryasoni98
requested review from
SamMorrowDrums,
juruen,
toby and
williammartin
as code owners
juruen
left a comment
juruen
left a comment
There was a problem hiding this comment.
Looking good! Just a little comment about making the test code more consistent with the existing code base.
juruen
left a comment
juruen
left a comment
There was a problem hiding this comment.
We are almost there! There's just a few lint errors that need to be fixed: we need to make sure we use github.Ptr and not the deprecated versions.
@juruen Update the code. Please review the PR . |
There was a problem hiding this comment.
@aryasoni98 I almost forget, we need to update the README file with the new tool! 🙇
juruen
left a comment
juruen
left a comment
There was a problem hiding this comment.
I believe we are just missing updating the README with the new tool
@juruen Update the README.md file |
juruen
left a comment
juruen
left a comment
There was a problem hiding this comment.
LGTM! 🚀
Thank you so much for your contribution!
* Add ability to view branches for a repo github#141 * fix: update ListBranches test to use InputSchema and correct translation helper * fix: update ListBranches test to use InputSchema and correct translation helper * fix: update ListBranches test to handle errors in tool result * fix: replace deprecated github.String with github.Ptr * docs: add list_branches tool documentation to README
4 participants
Closes: #141