Skip to content

MCP: Add example about PG-MCP #885

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
Apr 7, 2025
Merged

MCP: Add example about PG-MCP #885

merged 1 commit into from
Apr 7, 2025

Conversation

amotl
Copy link
Member

@amotl amotl commented Apr 3, 2025

About

References

@amotl amotl force-pushed the mcp-pg-mcp branch 5 times, most recently from 7ac95e8 to 285285c Compare April 3, 2025 21:56
@amotl amotl requested review from kneth and surister April 3, 2025 22:03
@amotl amotl marked this pull request as ready for review April 3, 2025 22:03
@stuzero
Copy link

stuzero commented Apr 4, 2025

Nice!

Comment on lines 1 to 18
#!/bin/sh
# Acquire MCP server part of `pg-mcp`.
# https://github.com/stuzero/pg-mcp
# https://github.com/crate-workbench/pg-mcp

# FIXME: Improve installation after packaging has been improved.
# https://github.com/stuzero/pg-mcp/issues/10

rm -rf pg-mcp
git clone --depth 1 --no-checkout --filter=blob:none \
https://github.com/crate-workbench/pg-mcp.git
cd pg-mcp
git checkout 16d7f61d5b3197777293ebae33b519f14a9d6e55 -- pyproject.toml uv.lock server test.py
uv pip install .
Copy link
Member Author

@amotl amotl Apr 4, 2025

Choose a reason for hiding this comment

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

Dear @stuzero,

thanks for your response, we appreciate it. We see room for improvements here, in order to get rid of this installation obstacle.

We will be happy to help, just let us know.

With kind regards,
Andreas.

@amotl
Copy link
Member Author

amotl commented Apr 4, 2025

@kneth: Thanks for acknowledging. This is yet another PR which is stacked on top of the other GH-880, so that one will need to be processed before. Thank you!

@amotl
Copy link
Member Author

amotl commented Apr 7, 2025

Thanks, @stuzero!

@amotl amotl merged commit 94a055f into main Apr 7, 2025
3 checks passed
@amotl amotl deleted the mcp-pg-mcp branch April 7, 2025 15:53
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.

3 participants