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: bump minimum Go version in go.mod to 1.21 #231

Merged
merged 1 commit into from
Aug 13, 2024

Conversation

alexsander-souza
Copy link
Contributor

No description provided.

Copy link
Member

@troyanov troyanov left a comment

Choose a reason for hiding this comment

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

TL;DR let's just bump min Go version (without toolchain directive)

FTR It seems we missed min Go version bump when updated bflad/tfproviderlint

go: github.com/bflad/tfproviderlint@v0.30.0 requires go@1.21, but 1.20 is requested

Setting the toolchain version to a different version than the min Go (language version) is useful in some rare cases. For example, if you want to compile the project with at least toolchain 1.22.6 but still want the language to be 1.21 (because for some reason you don't like the loop variable change).

So I think that in most cases there is no need to set toolchain version, as it's already implicitly set by the go directive.

go.mod Outdated Show resolved Hide resolved
@troyanov troyanov changed the title build: refresh go.mod chore: bump minimum Go version in go.mod to 1.21 Aug 13, 2024
@alexsander-souza alexsander-souza merged commit 5699b2e into canonical:master Aug 13, 2024
8 checks passed
@alexsander-souza alexsander-souza deleted the refresh_go_mod branch August 13, 2024 15:44
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