Skip to content

Conversation

@jamesmontemagno
Copy link

Updated the JSON configuration for GitHub integration in the README. Removed the old MCP structure and replaced it with a new format that includes server and input specifications.

Simplifies the directions. the "mcp" node isn't used anymore

Updated the JSON configuration for GitHub integration in the README. Removed the old MCP structure and replaced it with a new format that includes server and input specifications.
@jamesmontemagno jamesmontemagno requested a review from a team as a code owner November 6, 2025 22:13
Copilot AI review requested due to automatic review settings November 6, 2025 22:13
Copy link
Contributor

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 refactors the MCP server configuration documentation to eliminate redundancy and improve clarity. The main change consolidates duplicate JSON configuration examples into a single, reusable format.

Key Changes:

  • Updated the GitHub Enterprise Server/Cloud configuration example to use the standardized format with inputs and servers keys
  • Removed duplicate JSON configuration block that was shown for "other IDEs"
  • Removed the collapsible section showing the same example "without the MCP key"

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"args": [
"run",
"-i",
"--rm",
Copy link

Copilot AI Nov 6, 2025

Choose a reason for hiding this comment

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

The updated Docker command is missing the -e flags that were present in the original configuration. Without -e GITHUB_PERSONAL_ACCESS_TOKEN and -e GITHUB_HOST, the environment variables defined in the env block will not be passed to the Docker container.

Suggested change
"--rm",
"--rm",
"-e",
"GITHUB_PERSONAL_ACCESS_TOKEN",
"-e",
"GITHUB_HOST",

Copilot uses AI. Check for mistakes.
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