Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): update Go to 1.24 and switch to go-version-file #8388

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

knqyf263
Copy link
Collaborator

Overview

This PR updates Go to version 1.24 and changes how we manage Go versions in GitHub Actions from go-version to go-version-file.

Description

This PR includes the following changes:

  1. Update Go version to 1.24 in go.mod
  2. Bump golangci-lint to v1.64.2 that supports Go 1.24
  3. Switch from go-version to go-version-file in GitHub Actions workflows

The switch to go-version-file was made to improve version management clarity and reduce confusion. Previously, we had a discussion in #6711 about using go-version to automatically use the latest patch version. However, this approach led to confusion as the version specified in the workflow could differ from the one in go.mod.

For example, in #8341, we unnecessarily updated the Go version even though the latest patch version would have been used during the release anyway. To avoid such confusion, it looks better to manage Go versions explicitly through go.mod and use go-version-file in workflows. This approach:

  • Makes version management more transparent and explicit
  • Reduces the number of places where Go versions need to be managed
  • Ensures consistency between go.mod and CI/CD environments

Related PRs

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

This commit updates the Go version across multiple files from 1.23 to 1.24, including:
- .golangci.yaml
- Dockerfile.protoc
- go.mod
- GitHub workflow files
- Magefile golangci-lint version

Additionally, workflow files now use go-version-file instead of hardcoded GO_VERSION environment variable.
@knqyf263 knqyf263 self-assigned this Feb 12, 2025
@@ -1,4 +1,4 @@
FROM --platform=linux/amd64 golang:1.23
FROM --platform=linux/amd64 golang:1.24
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not released yet as of today.
https://hub.docker.com/_/golang/tags

Signed-off-by: knqyf263 <[email protected]>
@knqyf263
Copy link
Collaborator Author

Looks like TinyGo doesn't support Go 1.24.

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