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

Use go 1.23.3 #141

Merged
merged 4 commits into from
Nov 26, 2024
Merged

Use go 1.23.3 #141

merged 4 commits into from
Nov 26, 2024

Conversation

kke
Copy link
Contributor

@kke kke commented Nov 20, 2024

No description provided.

Signed-off-by: Kimmo Lehto <klehto@mirantis.com>
@kke kke added the chore label Nov 20, 2024
go.mod Outdated
@@ -1,8 +1,8 @@
module github.com/k0sproject/bootloose

go 1.22.0
go 1.23.0
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
go 1.23.0
go 1.23

Copy link
Contributor Author

@kke kke Nov 20, 2024

Choose a reason for hiding this comment

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

The version is now not pinned so binaries will be magically built and tests run with whatever 1.23 the CI has.

Maybe the version should be pinned in the workflow?

kke and others added 2 commits November 20, 2024 09:16
Co-authored-by: Tom Wieczorek <twz123@users.noreply.github.com>
Signed-off-by: Kimmo Lehto <klehto@mirantis.com>
@kke kke changed the title Use go 1.23.3 Use go 1.23 Nov 20, 2024
@kke
Copy link
Contributor Author

kke commented Nov 20, 2024

Maybe the toolchain should stay. Module consumers will not be affected by that and it would pin the CI tests and builds to a specific version without resorting to specifying it in each workflow yaml.

I believe this is how it is intended to be used.

Signed-off-by: Kimmo Lehto <klehto@mirantis.com>
@kke kke changed the title Use go 1.23 Use go 1.23.3 Nov 20, 2024
@kke kke requested a review from twz123 November 20, 2024 07:41
Copy link
Member

@twz123 twz123 left a comment

Choose a reason for hiding this comment

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

Yeah, I don't like that "let me auto-install the toolchain for you" thing and the "pinning to patch versions" in the mod file. I think the minimum supported Go version is good, but the actual toolchain should be at the caller's discretion 🤷 But what shall I say, Go decided differently and I have to remember to use GOTOOLCHAIN=local everywhere 😆

@twz123 twz123 merged commit 49d1249 into main Nov 26, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants