Skip to content

chore: document latest tag rather than main #77

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 1 commit into from
Apr 3, 2025

Conversation

SamMorrowDrums
Copy link
Collaborator

We now have a published latest docker tag that will keep up with actual releases, so we should avoid suggesting main as it is more likely to be broken or required further testing before rolling out.

@SamMorrowDrums SamMorrowDrums requested review from Copilot and toby and removed request for Copilot April 3, 2025 14:09
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 installation instructions in the README to remove the explicit reference to the "main" tag and rely on Docker's default behavior, which uses the published "latest" tag.

  • Changed docker command in the README to remove the ":main" tag.
  • Updated both code blocks where the image reference was specified.

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

Copy link
Collaborator

@williammartin williammartin left a comment

Choose a reason for hiding this comment

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

Seems good to me.

@SamMorrowDrums SamMorrowDrums merged commit bb56733 into main Apr 3, 2025
16 checks passed
@SamMorrowDrums SamMorrowDrums deleted the remove-docker-main branch April 3, 2025 14:20
manian0430 pushed a commit to ChrisLally/github-mcp-server that referenced this pull request Apr 12, 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.

2 participants