Skip to content

Conversation

@A11Might
Copy link
Contributor

Fix an example using mcp tag instead of jsonschema.

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, but don't edit this file. Edit the appropropriate .go file in internal/readme and run build.sh in that directory.

@A11Might
Copy link
Contributor Author

I tried to pull the latest commit, and running build.sh, there was no diff in README.md.

@A11Might A11Might requested a review from jba July 14, 2025 12:55
@jba
Copy link
Contributor

jba commented Jul 14, 2025

My mistake. You want to edit internal/readme/server/server.go. Then run the script.

@A11Might
Copy link
Contributor Author

Before I pull the latest commit and ran the script, I had already edited the internal/readme/server/server.go file in previous commit. And I didn't see any change in the README.md, I think the modification of README.md is the same as the script generation.

@jba
Copy link
Contributor

jba commented Jul 14, 2025

Ah yes, you're good. Thank you for this!

@jba jba merged commit bf7e518 into modelcontextprotocol:main Jul 14, 2025
4 checks passed
@A11Might A11Might deleted the fix-readme-example branch July 15, 2025 14:29
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