Skip to content

Conversation

@SamMorrowDrums
Copy link
Collaborator

The vscode install badge included main, this should now default to latest.

Copilot AI review requested due to automatic review settings April 4, 2025 14:31
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 updates the VS Code installation badge in the README to use the default Docker image tag (latest) instead of the fixed "main" tag.

  • Removed the explicit ":main" tag from the Docker image reference in the badge link.
  • Updated the installation URL to align with the intended default behavior.
Comments suppressed due to low confidence (1)

README.md:28

  • The updated Docker installation badge now references the Docker image without an explicit tag, which defaults to 'latest'. Please confirm that defaulting to 'latest' is the intended behavior, as it could lead to unpredictable deployments if the image updates.
+[<img alt="Install in VS Code" src="https://img.shields.io/badge/VS_Code-VS_Code?style=flat-square&label=Install%20Server&color=24bfa5">](https://vscode.dev/redirect?url=vscode%3Amcp%2Finstall%3F%25%7B%22name%22%3A%22github%22%2C%22command%22%3A%22docker%22%2C%22args%22%3A%5B%22run%22%2C%20%22-i%22%2C%20%22--rm%22%2C%20%22-e%22%2C%20%22GITHUB_PERSONAL_ACCESS_TOKEN%22%2C%20%22ghcr.io%2Fgithub%2Fgithub-mcp-server%22%5D%2C%20%22env%22%3A%20%7B%22GITHUB_PERSONAL_ACCESS_TOKEN%22%3A%20%22%24%7Binput%3Agithub-pat%7D%22%7D%2C%20%22inputs%22%3A%20%5B%7B%20%22id%22%3A%20%22github-pat%22%2C%20%22type%22%3A%20%22promptString%22%2C%20%22description%22%3A%20%22Github%20Personal%20Access%20Token%22%2C%20%22password%22%3A%20true%7D%5D%7D)

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

@SamMorrowDrums SamMorrowDrums changed the title chode: remove main docker tag from README.md chore: remove main docker tag from README.md Apr 4, 2025
DaleSeo pushed a commit to DaleSeo/github-mcp-server that referenced this pull request Oct 24, 2025
* Add Dynatrace MCP Server

* fix typo

* Update servers/dynatrace-mcp-server/server.yaml

Co-authored-by: Cody Rigney <cody.rigney@docker.com>

---------

Co-authored-by: Cody Rigney <cody.rigney@docker.com>
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