Skip to content

use go 1.26#128

Merged
moleske merged 2 commits into
masterfrom
moleske-patch-1
Feb 12, 2026
Merged

use go 1.26#128
moleske merged 2 commits into
masterfrom
moleske-patch-1

Conversation

@moleske

@moleske moleske commented Feb 12, 2026

Copy link
Copy Markdown
Member

1.24 is out of support

1.24 is out of support
@moleske moleske requested review from a team February 12, 2026 02:37

@blgm blgm 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.

LGTM

Just a note that you don't have to update the Go directive just because the version mentioned is out of support. As long as the Go compiler is in support, then everything should work in a backwards-compatible way.

Having said this, I like updating the Go directive as it prevents accidentally using an old compiler, and opts in to new features.

@jcvrabo jcvrabo 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.

nothing much to add, as it has already been approved and have nothing against it (good to keep the house clean)

@moleske

moleske commented Feb 12, 2026

Copy link
Copy Markdown
Member Author

Just a note that you don't have to update the Go

I did remember this, though we've also configured the github action release and ci use the go version from the file. So seems worth keeping that up to date so that the github action uses a reasonable version of go. Rather mange the go version in one place rather than all three while making sure our tests/ci system use a supported go version

@moleske moleske merged commit 34a8d1c into master Feb 12, 2026
2 checks passed
@moleske moleske deleted the moleske-patch-1 branch February 12, 2026 15:55
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