-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Do not use toolchain directive #5660
Comments
is this already fixed and merged? |
From issue description:
|
With the upgrade of Go to 1.23, the use of |
Closing this, as with Go 1.23, the toolchain directive is required and added automatically. |
It is still not required. See: #5784 There are more people complaining on the |
I think that
go 1.21.0
is required for the bump and it should not be a problem..However, I do not think we need to add
toolchain go1.22.5
. I think it would be better to avoid toolchain consistently in all our Go modules.Originally posted by @pellared in #5643 (comment)
We should have have some "linter" that would detect such changes.
The text was updated successfully, but these errors were encountered: