Skip to content

Conversation

@cr2007
Copy link
Contributor

@cr2007 cr2007 commented Jul 18, 2025

This PR improves the formatting and internal link structure of both the main README.md and the source file at internal/readme/README.src.md. Changes include:

🛠 Changes Made

  • Replaced plain-text warning with a GitHub-style admonition block (> [!WARNING]) for improved visibility and consistency with other docs.

Example:

Warning

The SDK should be considered unreleased, and is currently unstable
and subject to breaking changes. Please test it out and file bug reports or API
proposals, but don't use it in real projects. See the issue tracker for known
issues and missing features. We aim to release a stable version of the SDK in
August, 2025.

  • Updated markdown links to use cleaner relative paths:
    • Changed CONTRIBUTING.md reference to [CONTRIBUTING.md](/CONTRIBUTING.md)
    • Updated examples/ reference to [examples/](/examples/)
  • Ensured that all updates were mirrored in both the generated and source README files.

📌 Why It Matters

These changes improve the readability and usability of the documentation, especially when viewed on GitHub. The updated formatting helps call attention to important warnings, and fixing relative paths ensures that links work reliably across platforms and contexts.

cr2007 added 2 commits July 18, 2025 09:15
* Converted warning section to GitHub-style admonition block using
  `> [!WARNING]` for better visual clarity
* Updated relative links for `CONTRIBUTING.md` and `examples/` directory
  to use markdown link syntax with cleaner URLs
* Applied the same changes to both `README.md` and the source version at
  `internal/readme/README.src.md` to maintain consistency
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 d437c81 into modelcontextprotocol:main Jul 18, 2025
@cr2007 cr2007 deleted the readme-fix branch July 18, 2025 13:19
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