Skip to content

docs: update documentation on mcp usage#93

Merged
sgoedecke merged 4 commits intoactions:mainfrom
FidelusAleksander:main
Aug 26, 2025
Merged

docs: update documentation on mcp usage#93
sgoedecke merged 4 commits intoactions:mainfrom
FidelusAleksander:main

Conversation

@FidelusAleksander
Copy link
Contributor

Fixes: #92

Copilot AI review requested due to automatic review settings August 15, 2025 05:53
@FidelusAleksander FidelusAleksander requested a review from a team as a code owner August 15, 2025 05:53
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

Updates documentation to clarify GitHub MCP (Model Context Protocol) server authentication requirements and security recommendations.

  • Clarifies that GitHub MCP requires a Personal Access Token instead of the built-in GITHUB_TOKEN
  • Adds security recommendation to use a separate PAT for tighter security
  • Includes a note section highlighting the PAT requirement

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
action.yml Updates description for github-mcp-token parameter to clarify token fallback behavior and recommend separate PAT
README.md Adds note section emphasizing PAT requirement for GitHub MCP integration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

sgoedecke
sgoedecke previously approved these changes Aug 15, 2025
@sgoedecke
Copy link
Contributor

One more thing - would you mind running npx prettier --write . to fix the Markdown linting errors?

@FidelusAleksander
Copy link
Contributor Author

Done @sgoedecke

@sgoedecke sgoedecke merged commit 83bb5ca into actions:main Aug 26, 2025
6 checks passed
@sgoedecke
Copy link
Contributor

Thank you!

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.

[BUG]: MCP Usage Fails

3 participants