- Notifications
You must be signed in to change notification settings - Fork898
update install instructions#43
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
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.
Pull Request Overview
This PR updates the install instructions in the README to include a Docker-based installation option as the primary method, while also providing steps for building the binary from the repo. Key changes include the addition of Docker image requirements and updated configuration instructions for starting VS Code Insiders.
Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab.Learn more
Uh oh!
There was an error while loading.Please reload this page.
42b407c toc402539Comparec402539 tocda36e8CompareThere 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.
Pull Request Overview
This PR updates the install instructions in the README to reflect the new installation options for VS Code Insiders, with a focus on the Docker image option and the removal of variable token inputs.
- Added a Docker image installation section with authentication instructions.
- Updated the build-from-repo instructions and removed the token input configuration for the server setup.
Comments suppressed due to low confidence (2)
README.md:414
- Consider aligning the server naming keys across both configuration options. For instance, using a consistent key (e.g., 'github-mcp-server') in both the Docker and binary sections may improve clarity for users.
"github-mcp-server": {README.md:412
- Since the previous token input configuration was removed, consider adding an inline comment to clarify that this change is intentional due to variable input limitations. This may prevent future confusion when users expect a token prompt.
"inputs": [],Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused.Learn more
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 hope the input access is fixed soon 😅
Uh oh!
There was an error while loading.Please reload this page.
aa1d72d to6dbe595Compare09c9799 intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Context
Update doc on how to install and test this out in VS insiders. Show Docker option as the first choice to do so.
As variable inputs don't seem to work to introduce the token, we remove that from the README until they work.