-
Notifications
You must be signed in to change notification settings - Fork 255
Blog post and documentation changes for Remote MCP #16168
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
Conversation
Your site preview for commit b2acc2f is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16168-b2acc2f6.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit 471bf06 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16168-471bf06b.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit 8714e49 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16168-8714e497.s3-website.us-west-2.amazonaws.com. |
@claude review this PR for technical accuracy and content. |
Your site preview for commit 94aa0e2 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16168-94aa0e2e.s3-website.us-west-2.amazonaws.com. |
Your site preview for commit d133bf4 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16168-d133bf4d.s3-website.us-west-2.amazonaws.com. |
@claude review this PR for technical accuracy and content. |
Claude finished @arturl's task —— View job PR Review: Remote MCP ChangesReviewing this PR for technical accuracy and content quality. Tasks
Technical accuracyNew blog post (
|
Pull Request ReviewThank you for this comprehensive update on the Remote MCP Server! Overall, this is strong content with good structure and clear explanations. I've identified several issues that need attention before merging. Critical Issues1. Typo in PR descriptionIssue: "Changs for remote MCP" should be "Changes for remote MCP" 2. Missing newline at end of fileFile: content/blog/remote-mcp-server/index.md (line 194) 3. Incomplete file in documentationFile: content/docs/iac/using-pulumi/mcp-server/index.md (line 85) Style and Content Issues4. Vague languageFile: content/blog/remote-mcp-server/index.md (lines 21, 25) 5. Avoid "simply"File: content/blog/remote-mcp-server/index.md (line 35) 6. Passive voiceFile: content/blog/remote-mcp-server/index.md (line 49) 7. Run-on sentenceFile: content/blog/remote-mcp-server/index.md (line 154) 8. Inconsistent code formattingFile: content/blog/remote-mcp-server/index.md (lines 65-67, 71-72) 9. Repetitive structureFile: content/blog/remote-mcp-server/index.md (lines 120-147) 10. Unclear instructionsFile: content/docs/iac/using-pulumi/mcp-server/index.md (lines 67-71) Recommendations11. Verify linksPlease confirm these internal links work:
12. Verify imageFile: content/docs/iac/using-pulumi/mcp-server/index.md (line 88) 13. Alias correctly addedGood work: The alias for the moved file is correctly added (line 13-14). SummaryMust Fix:
Strongly Recommended:
Overall, this is excellent content that will be helpful for users. Once the critical issues are addressed, this will be ready to merge! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only reviewed the blog post. I'll defer the docs to @CamSoper
Your site preview for commit cbf00a5 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16168-cbf00a52.s3-website.us-west-2.amazonaws.com. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No notes, LGTM!
Blog post announcing Remote MCP server and changes in the docs.