Skip to content

Update README.md #189

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

Merged
merged 1 commit into from
Jul 2, 2025
Merged

Update README.md #189

merged 1 commit into from
Jul 2, 2025

Conversation

leestott
Copy link
Contributor

@leestott leestott commented Jul 2, 2025

Update intro with Mermaid and link to MCP protocol and specification change log

Describe the intention of the changes being proposed. What problem does it solve or functionality does it add?

Updating details and references

Does this introduce a breaking change?

When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.

[ ] Yes
[x] No

Does this require changes to learn.microsoft.com docs or modules?

which includes deployment, settings and usage instructions.

[ ] Yes
[x] No

Type of change

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

Update intro with Mermaid and link to MCP protocol and specification change log
@Copilot Copilot AI review requested due to automatic review settings July 2, 2025 06:44
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the MCP Core Concepts README by adding hyperlinks, a visual diagram, and a link to the latest protocol specification.

  • Turned the introductory sentence into a linked reference to the Model Context Protocol GitHub organization
  • Inserted a Mermaid flowchart illustrating the MCP client-server architecture
  • Added a link to the 2025-06-18 protocol specification change log
Comments suppressed due to low confidence (3)

01-CoreConcepts/README.md:35

  • [nitpick] The backslash in Local\Data Source A may be unintended and could confuse the diagram. Use Local Data Source A or a forward slash for clarity.
        S1 <--> D1[("Local\Data Source A")]

01-CoreConcepts/README.md:49

  • [nitpick] For improved readability, consider adding a comma or semicolon after 'standard', e.g., 'evolving standard; you can see...'.
The MCP Protocol is an evolving standard you can see the latest updates to the [protocol specification](https://modelcontextprotocol.io/specification/2025-06-18/)

01-CoreConcepts/README.md:28

  • [nitpick] The label has an unmatched parenthesis and 'VScode' is misspelled. Consider changing to something like Host['Host (with MCP, VS Code, IDEs, Tools)'].
        Host["Host with MCP VScode, IDEs, Tools)"]

@leestott leestott merged commit e6b0156 into main Jul 2, 2025
7 checks passed
@leestott leestott deleted the leestott-patch-7 branch July 2, 2025 07:48
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.

1 participant