Skip to content

fix(docker): align Go toolchain and validate multi-platform images - #267

Merged
Zxilly merged 2 commits into
masterfrom
fix/docker-go-toolchain
Oct 8, 2026
Merged

Zxilly merged 2 commits into
masterfrom
fix/docker-go-toolchain

Conversation

@Zxilly

@Zxilly Zxilly commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Align the Docker builder with the Go 1.24.1 toolchain already declared in go.mod.
  • Build linux/amd64 and linux/arm64/v8 images in pull request CI without publishing or registry credentials.

Why

The post-merge image build for #266 fails because Go 1.22.12 cannot build the module, which now requires Go >= 1.23.0.
Failure: https://github.com/ZNotify/server/actions/runs/37629925018/job/112822742458

Validation

  • Workflow YAML parsed and architecture/no-publish assertions passed.
  • git diff --check passed.
  • All 30 PR CI jobs passed on bafaae6: lint, server unit tests, three-platform binaries, 24 SDK integration jobs, and the amd64/arm64 Docker build.
  • CI: https://github.com/ZNotify/server/actions/runs/37737456830
  • Codacy and the additional summary check passed; Mergify reported neutral. No failing or pending checks remain.

Zxilly added 2 commits October 7, 2026 23:22
Added a new job to build Docker images for amd64 and arm64 architectures in the pull request workflow.
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@Zxilly
Zxilly marked this pull request as ready for review October 8, 2026 06:50
@mergify

mergify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@Zxilly
Zxilly merged commit 6a88983 into master Oct 8, 2026
33 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.

1 participant