Skip to content

chore: use go1.17 in go.mod #2282

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

Closed
wants to merge 1 commit into from
Closed

chore: use go1.17 in go.mod #2282

wants to merge 1 commit into from

Conversation

ernado
Copy link
Member

@ernado ernado commented Oct 8, 2021

This allows lazy loading.

This allows lazy loading.
@ldez ldez self-requested a review October 8, 2021 07:12
@ldez
Copy link
Member

ldez commented Oct 8, 2021

I'm not sure about this change, because it will "allow" using go1.17 elements in the code, and this can be a problem for people that use the "tools" pattern (use an import to get the tools) to get golangci-lint.

Related to #2183

@ernado
Copy link
Member Author

ernado commented Oct 8, 2021

Should we wait until go1.18 then?

@ldez
Copy link
Member

ldez commented Oct 8, 2021

Yes I think it can be better to wait for the next release of Go (1.18)

@ernado
Copy link
Member Author

ernado commented Oct 8, 2021

Closing for now, will not move to 1.17 until 1.18 is released.

@ernado ernado closed this Oct 8, 2021
@ldez ldez added the declined label Oct 8, 2021
@SVilgelm SVilgelm deleted the chore/go-mod-use-1-17 branch October 9, 2021 01:36
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