Skip to content

fix: Update Dockerfile to go 1.25#817

Merged
Dumbris merged 1 commit into
smart-mcp-proxy:mainfrom
Eun:patch-1
Jul 8, 2026
Merged

fix: Update Dockerfile to go 1.25#817
Dumbris merged 1 commit into
smart-mcp-proxy:mainfrom
Eun:patch-1

Conversation

@Eun

@Eun Eun commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Pull Request

Description

The go.mod file uses go 1.25, update the Dockerfile so it also uses go 1.25 for building.

Testing

  • I have tested these changes locally
  • I have added/updated tests that prove my fix is effective or my feature works
  • All existing tests pass

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Dumbris Dumbris left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Trivial and correct: bumps the Docker builder image from golang:1.24-alpine to golang:1.25-alpine to match go 1.25.5 in go.mod. All required CI checks pass. (The auto-pass-qa-gate job errored with HTTP 403 because fork PRs receive a read-only GITHUB_TOKEN that can't POST commit statuses — the qa-gate status was posted manually; worth fixing the workflow to use pull_request_target or handle fork PRs so this doesn't recur.)

@Dumbris Dumbris merged commit dd1b99c into smart-mcp-proxy:main Jul 8, 2026
34 of 35 checks passed
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