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

Bump golangci-ling to 1.51.2 #227

Merged
merged 1 commit into from
Mar 7, 2023
Merged

Bump golangci-ling to 1.51.2 #227

merged 1 commit into from
Mar 7, 2023

Conversation

vkhoroz
Copy link
Member

@vkhoroz vkhoroz commented Mar 6, 2023

I'm helping with LmP upgrade of Golang to 1.20 for fioconfig. So, my system-wide Golang version was upgraded to version 1.20. But, for some weird reason, older golangci-lint stopped working after that regardless of the go1.19 flag in go.mod.
It looks like it uses some of the Golang internals. The easiest solution is to upgrade golangci-lint to a newer version, which support Golang 1.17+.

Looking ahead, this is a good addition anyway to stay tuned.

Signed-off-by: Volodymyr Khoroz volodymyr.khoroz@foundries.io

I'm helping with LmP upgrade of Golang to 1.20 for fioconfig.
So, my system-wide Golang version was upgraded to version 1.20.
But, for some weird reason, older golangci-lint stopped working after that
regardless of the go1.19 flag in go.mod.
It looks like it uses some of the Golang internals.
The easiest solution is to upgrade golangci-lint to a newer version,
which support Golang 1.17+.

Looking ahead, this is a good addition anyway to stay tuned.

Signed-off-by: Volodymyr Khoroz <volodymyr.khoroz@foundries.io>
@vkhoroz vkhoroz requested a review from doanac March 6, 2023 19:10
@vkhoroz vkhoroz self-assigned this Mar 6, 2023
@vkhoroz
Copy link
Member Author

vkhoroz commented Mar 6, 2023

Maybe, these tickets help:

In short: we need to upgrade golint every time Golang releases a new version and we plan to support it.

@vkhoroz vkhoroz merged commit 31d4720 into main Mar 7, 2023
@vkhoroz vkhoroz deleted the vkhoroz-golint-1.51.2 branch March 7, 2023 13:51
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