Skip to content

Conversation

tonytrg
Copy link
Contributor

@tonytrg tonytrg commented Apr 11, 2025

This pr adds:

  • excludes a few paths for go binaries
  • adds a bit more context for the Prerequisites section in the readme.

@Copilot Copilot AI review requested due to automatic review settings April 11, 2025 12:25
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 introduces minor documentation updates to improve setup instructions.

  • Added a note specifying that the Personal Access Token requires at least the read:packages permission.
  • Added a step for logging into ghcr.io using docker.

Co-authored-by: Copilot <[email protected]>
README.md Outdated
@@ -18,7 +18,8 @@ automation and interaction capabilities for developers and tools.
2. Once Docker is installed, you will also need to ensure Docker is running.
3. Lastly you will need to [Create a GitHub Personal Access Token](https://github.com/settings/personal-access-tokens/new).
The MCP server can use many of the GitHub APIs, so enable the permissions that you feel comfortable granting your AI tools (to learn more about access tokens, please check out the [documentation](https://docs.github.com/en/authentication/keeping-your-account-and-data-secure/managing-your-personal-access-tokens)).

* Your PAT Token needs at least `read:packages` permission to be able to pull from `ghcr.io`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you run into some issue here? As far as I'm aware the image is public and:

You can also access public container images anonymously.

https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry#about-the-container-registry

Copy link
Contributor Author

@tonytrg tonytrg Apr 11, 2025

Choose a reason for hiding this comment

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

ah whoops ignore this pr for now :)

I just tested and it works without the pat.

I had some issues with the setup earlier. So I wiped everything and go through every option in the readme to check if I can add useful stuff to the readme.

@tonytrg tonytrg marked this pull request as draft April 11, 2025 12:43
@tonytrg tonytrg closed this Apr 11, 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