Skip to content

chore: remove main docker tag from README.md #100

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

Closed
wants to merge 1 commit into from

Conversation

SamMorrowDrums
Copy link
Collaborator

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

@Copilot Copilot AI review requested due to automatic review settings April 4, 2025 14:31
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 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
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.

1 participant