Skip to content

ci: Use local mcp-publisher binary for login#47

Merged
masnwilliams merged 2 commits intomainfrom
mason/ga-release
Sep 10, 2025
Merged

ci: Use local mcp-publisher binary for login#47
masnwilliams merged 2 commits intomainfrom
mason/ga-release

Conversation

@masnwilliams
Copy link
Copy Markdown
Collaborator

@masnwilliams masnwilliams commented Sep 10, 2025

TL;DR

Updated the publish-mcp workflow to explicitly use the local ./mcp-publisher binary for login.

Why we made these changes

The previous command relied on the mcp-publisher executable being in the system's PATH, which can be unreliable in CI environments. This change ensures the workflow uses the specific binary from the current directory, preventing potential path issues or conflicts with other installed versions.

What changed?

  • .github/workflows/publish-mcp.yml: The login command was updated from mcp-publisher to ./mcp-publisher to ensure the local executable is used.

Description generated by Mesa. Update settings

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
mcp Ready Ready Preview Comment Sep 10, 2025 9:45pm

@masnwilliams masnwilliams merged commit 2823daa into main Sep 10, 2025
1 of 2 checks passed
@masnwilliams masnwilliams deleted the mason/ga-release branch September 10, 2025 21:43
Copy link
Copy Markdown

@mesa-dot-dev mesa-dot-dev Bot left a comment

Choose a reason for hiding this comment

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

Performed full review of b16ecc4...8ae8804

Tip

⚡ Quick Actions

This review was generated by Mesa.

Actions:

Slash Commands:

  • /review - Request a full code review
  • /review latest - Review only changes since the last review
  • /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
  • /help - Get help with Mesa commands and configuration options

1 files reviewed | 0 comments | Review on Mesa | Edit Reviewer Settings

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