Skip to content

Comments

Add MCP config#278

Merged
lkostrowski merged 1 commit intomainfrom
mcp
May 9, 2025
Merged

Add MCP config#278
lkostrowski merged 1 commit intomainfrom
mcp

Conversation

@lkostrowski
Copy link
Member

No description provided.

@lkostrowski lkostrowski requested a review from a team May 9, 2025 10:02
@lkostrowski lkostrowski requested review from Copilot and kzuraw and removed request for a team May 9, 2025 10:03
Copy link

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 adds a new MCP configuration file to support the Saleor MCP server setup for different environments.

  • Added a JSON configuration file in both the .vscode and .cursor directories.
  • Configures MCP server details including command, arguments, and environment variables.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
.vscode/mcp.json Introduces MCP server configuration for VSCode.
.cursor/mcp.json Introduces MCP server configuration for Cursor.

@@ -0,0 +1,11 @@
{
Copy link

Copilot AI May 9, 2025

Choose a reason for hiding this comment

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

[nitpick] The MCP configuration appears to be duplicated in both the .vscode and .cursor directories. Consider centralizing the configuration to reduce duplication and potential inconsistencies.

Copilot uses AI. Check for mistakes.
@@ -0,0 +1,11 @@
{
Copy link

Copilot AI May 9, 2025

Choose a reason for hiding this comment

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

[nitpick] The MCP configuration appears to be duplicated in both the .cursor and .vscode directories. Consider centralizing the configuration to reduce duplication and potential inconsistencies.

Copilot uses AI. Check for mistakes.
@lkostrowski lkostrowski merged commit 2aa587b into main May 9, 2025
6 checks passed
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