Skip to content

Conversation

@johns10
Copy link

@johns10 johns10 commented Mar 3, 2025

Nice library. I noticed in the docs you mentioned supporting the hosting model, but there was no RunAsync method defined. So I added it. See if it meets your approval.

@salty-flower
Copy link
Owner

Hi John, it's great to see people opening PRs!
However, I think I've published the ModelContextProtocol.NET.Server.Hosting which includes a async wrapper method. May I know why that didn't work or why you think putting into server package is a better design?

Regarding the content itself - I saw a lot of noise by formatting. IMO The style itself doesn't matter but remaining consistent (so that no noise from styling) matters. I've been using CSharpier.

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