Skip to content

Conversation

@qt-luigi
Copy link
Contributor

@qt-luigi qt-luigi commented Jul 13, 2025

jsonrpc's github path notation and pkg.go.dev link url are broken.

jsonrpc's github path notation and pkg.go.dev link url are broken.
@jba
Copy link
Contributor

jba commented Jul 14, 2025

Thanks! But don't edit README.md, edit internal/readme/README.src.md and then run build.sh from that directory.

Copy link
Contributor

@jba jba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

edit internal/readme/README.src.md and run build.sh

@qt-luigi qt-luigi requested a review from jba July 14, 2025 11:54
@qt-luigi
Copy link
Contributor Author

Thank you and sorry jba. I retry it.

Copy link
Contributor

@jba jba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see README.src.md in your PR.

jsonrpc's github path notation and pkg.go.dev link url are broken.
Copy link
Contributor

@jba jba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jba jba merged commit b780f1b into modelcontextprotocol:main Jul 14, 2025
4 checks passed
@qt-luigi
Copy link
Contributor Author

I added internal/readme/README.src.md and ran ./build.sh on my PC and confirmed there were no errors.

@qt-luigi qt-luigi deleted the qt-luigi-patch-1 branch July 14, 2025 12:30
@qt-luigi
Copy link
Contributor Author

Thanks for merging me! I am looking forward to this go-sdk becoming GA!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants