Skip to content

Comments

build: update to Go 1.26#65

Merged
tembleking merged 1 commit intomainfrom
build/update-go-1-26
Feb 18, 2026
Merged

build: update to Go 1.26#65
tembleking merged 1 commit intomainfrom
build/update-go-1-26

Conversation

@tembleking
Copy link
Member

  • Add the new go fix tool to automatically modernize code to newer versions.
  • Update dependencies.
  • Update base images.
  • Build with the newer 1.26 toolchain.

@tembleking tembleking self-assigned this Feb 18, 2026
Copilot AI review requested due to automatic review settings February 18, 2026 10:54
@tembleking tembleking requested a review from a team as a code owner February 18, 2026 10:54
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

This pull request updates the project from Go 1.24 to Go 1.26, leveraging new language features and updating dependencies. The update introduces support for the new go fix tool and adopts Go 1.26's enhanced new() syntax that appears to accept value expressions directly, eliminating the need for custom pointer helper functions.

Changes:

  • Upgraded Go toolchain from 1.24 to 1.26 across build configurations
  • Replaced custom toPtr/asPtr helper functions with Go 1.26's new(value) syntax throughout the codebase
  • Added go fix command integration to the build pipeline and pre-commit hooks
  • Updated dependencies (mcp-go, ginkgo, gomega, golang.org/x packages) and base Docker images

Reviewed changes

Copilot reviewed 22 out of 24 changed files in this pull request and generated no comments.

Show a summary per file
File Description
package.nix Updated to use buildGo126Module and refreshed vendorHash for new dependencies
go.mod Updated Go version to 1.26.0 and bumped dependency versions
go.sum Updated checksums for new dependency versions
flake.nix Updated to use go_1_26 in dev environment and removed old go override
flake.lock Updated nixpkgs to newer revision
justfile Added fix command using go fix ./... and integrated it into check pipeline
.pre-commit-config.yaml Added pre-commit hook for go fix
.golangci.yml Created linter config disabling SA4006/SA4017 for Go 1.26 new() syntax compatibility
docker-base-*.nix Updated base image digests and hashes
internal/infra/mcp/tools/utils.go Removed custom toPtr helper function
internal/infra/mcp/tools/tool_list_runtime_events.go Migrated from toPtr() to new() syntax
internal/infra/mcp/tools/*_test.go Migrated from toPtr()/asPtr() to new() syntax in test files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@alecron alecron left a comment

Choose a reason for hiding this comment

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

LGTM

@tembleking tembleking enabled auto-merge (squash) February 18, 2026 11:01
@tembleking tembleking merged commit b67cf70 into main Feb 18, 2026
12 checks passed
@tembleking tembleking deleted the build/update-go-1-26 branch February 18, 2026 11:12
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.

2 participants