Skip to content

update Go versions#92

Merged
flyingmutant merged 1 commit intoflyingmutant:masterfrom
conetcloud:m.gass/ci_add_go_1_25_and_1_26_xkopnxomxpos
Mar 30, 2026
Merged

update Go versions#92
flyingmutant merged 1 commit intoflyingmutant:masterfrom
conetcloud:m.gass/ci_add_go_1_25_and_1_26_xkopnxomxpos

Conversation

@mxey
Copy link
Copy Markdown
Contributor

@mxey mxey commented Mar 30, 2026

Remove lower than 1.23, because the go.mod requires 1.23.
If running on lower versions, the go command just downloaded 1.23 in CI anyway.

Add newer versions 1.25 and 1.26

@mxey mxey marked this pull request as draft March 30, 2026 10:09
@mxey mxey force-pushed the m.gass/ci_add_go_1_25_and_1_26_xkopnxomxpos branch 2 times, most recently from 1e76063 to d05e03a Compare March 30, 2026 10:14
@mxey mxey changed the title ci: add Go 1.25 and 1.26 ci: update Go versions Mar 30, 2026
@mxey
Copy link
Copy Markdown
Contributor Author

mxey commented Mar 30, 2026

I guess we could also remove strings_example_test.go, since that only runs on Go < 1.21

@mxey mxey marked this pull request as ready for review March 30, 2026 10:17
@flyingmutant
Copy link
Copy Markdown
Owner

I guess we could also remove strings_example_test.go, since that only runs on Go < 1.21

Sounds good (strings_example_go121_test should just become strings_example_test).

@mxey mxey force-pushed the m.gass/ci_add_go_1_25_and_1_26_xkopnxomxpos branch from d05e03a to ecd5255 Compare March 30, 2026 13:14
@mxey mxey changed the title ci: update Go versions update Go versions Mar 30, 2026
@mxey
Copy link
Copy Markdown
Contributor Author

mxey commented Mar 30, 2026

@flyingmutant applied.

@flyingmutant
Copy link
Copy Markdown
Owner

Looks good, please rebase.

Remove lower than 1.23, because the go.mod requires 1.23.
If running on lower versions, the go command just downloaded 1.23 in CI anyway.

Add newer versions 1.25 and 1.26

Remove obsolete code for Go < 1.21.
@mxey mxey force-pushed the m.gass/ci_add_go_1_25_and_1_26_xkopnxomxpos branch from ecd5255 to 1b279f4 Compare March 30, 2026 14:46
@mxey
Copy link
Copy Markdown
Contributor Author

mxey commented Mar 30, 2026

@flyingmutant done.

@flyingmutant flyingmutant merged commit 9bafe07 into flyingmutant:master Mar 30, 2026
14 checks passed
@flyingmutant
Copy link
Copy Markdown
Owner

Thanks!

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