Skip to content

Simplify vscode MCP setup with input #19

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 2 commits into from
Mar 20, 2025
Merged

Simplify vscode MCP setup with input #19

merged 2 commits into from
Mar 20, 2025

Conversation

connor4312
Copy link
Contributor

note: realized this doesn't actually work (microsoft/vscode#243875) but I will fix that today and wait to merge until then

note: realized this doesn't actually work (microsoft/vscode#243875) but I will fix that today and wait to merge until then
@Copilot Copilot AI review requested due to automatic review settings March 18, 2025 15:15
Copy link
Contributor

@Copilot 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 aims to simplify the vscode MCP setup by replacing manual environment variable configuration with an input-based setup using VS Code settings.

  • Updated installation instructions for the MCP configuration
  • Modified the JSON configuration to prompt for the GitHub Personal Access Token instead of directly setting an environment variable

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@SamMorrowDrums
Copy link
Collaborator

@connor4312 if you're happy to resolve conflicts and merge that would be awesome, if you want me to do so give me a ping! 🔔

@connor4312 connor4312 merged commit 14fb02b into main Mar 20, 2025
3 checks passed
manian0430 pushed a commit to ChrisLally/github-mcp-server that referenced this pull request Apr 12, 2025
note: realized this doesn't actually work (microsoft/vscode#243875) but I will fix that today and wait to merge until then
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