Skip to content

Draft Building an MCP server for your FastAPI API with FastMCP#20

Merged
sixhobbits merged 22 commits into
mainfrom
draft/building-mcp-server-with-fastapi-with-fastmcp
Jul 1, 2025
Merged

Draft Building an MCP server for your FastAPI API with FastMCP#20
sixhobbits merged 22 commits into
mainfrom
draft/building-mcp-server-with-fastapi-with-fastmcp

Conversation

@koladev32
Copy link
Copy Markdown
Collaborator

No description provided.

@ritza-publishing-assistant
Copy link
Copy Markdown

koladev32 and others added 5 commits June 5, 2025 14:37
…to draft/qa-mcp-server-with-fastapi-with-fastmcp

# Conflicts:
#	mcp/building-mcp-server-for-fastapi-with-fastmcp.mdx
@rideam rideam added the qa-changes-requested Something isn't working label Jun 9, 2025
rideam and others added 6 commits June 9, 2025 11:01
…to draft/qa-mcp-server-with-fastapi-with-fastmcp

# Conflicts:
#	mcp/building-mcp-server-for-fastapi-with-fastmcp.mdx
…-with-fastmcp

qa: MCP server with fastapi with fastmcp
@ritza-publishing-assistant
Copy link
Copy Markdown

https://github.com/ritza-co/speakeasy-developer-docs/blob/draft/building-mcp-server-with-fastapi-with-fastmcp/mcp/building-mcp-server-for-fastapi-with-fastmcp.mdx (

  • Introduction paragraph: (Apitizing Burgers API project vs. apitizing-burger project) Inconsistent naming/pluralization (“Apitizing Burgers API” is used in one instance while “apitizing-burger” is used in another).
  • Configuration section: (But, fastmcp is not the only one dependency) Awkward phrasing; consider “fastmcp is not the only dependency” instead of “one dependency” for grammatical correctness.
  • Code snippet (mcp_server.py): (if name == "main": mcp.run()) The code block uses two-space indentation, which may be inconsistent with common Python indentation practices though this might be intentional.)

@sixhobbits sixhobbits merged commit cdc7122 into main Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants