Skip to content

Conversation

MattBabbage
Copy link
Contributor

@MattBabbage MattBabbage commented Sep 26, 2025

This pull request updates the release workflow and configuration to improve version management, automate validation, and align metadata with MCP requirements. The most important changes are grouped below.

All PRs will be squashed into one, lots of trial and error took place.

@MattBabbage MattBabbage requested a review from a team as a code owner September 26, 2025 09:21
@Copilot Copilot AI review requested due to automatic review settings September 26, 2025 09:21
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 pull request updates the release workflow and configuration to improve version management, automate validation, and align metadata with MCP requirements.

  • Adds MCP server label to Docker container and uses dynamic versioning
  • Improves the registry release workflow with Go setup, tag fetching, and validation
  • Removes hardcoded version strings and unused remote configuration

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
server.json Updates version placeholders and package identifier, removes unused remotes section
Dockerfile Adds MCP server annotation label for proper identification
.github/workflows/registry-releaser.yml Enhances workflow with Go setup, improved publisher installation, and configuration validation

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

@MattBabbage MattBabbage merged commit 35b0da2 into main Sep 26, 2025
16 checks passed
@MattBabbage MattBabbage deleted the matt/publish-to-mcp-registry branch September 26, 2025 11:08
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.

3 participants