Specify golangci-lint version supporting Go 1.23
#114
Conversation
|
golangci-lint 도 golang 신규 버전을 비교적 빨리 받아들이는 듯 한데, golangci-lint 버전을 명시해서 사용하는 것은 어떠실까요? 반대로 현 pr 제안과 같이 latest로 하고, 오류가 발생하면(golangci-lint가 더 높은 golang버전을 쓰게 되면) |
There was a problem hiding this comment.
@ish-hcc Linter 부분을 꼼꼼히 챙겨주셔서 감사드립니다.
수정 제안 드린 부분을 확인해보시고, 이슈가 있으시면 공유하여 주시기 바랍니다.
Makefile
Outdated
| @@ -21,7 +21,7 @@ dependency: ## Get dependencies | |||
| lint: dependency ## Lint the files | |||
| @echo "Running linter..." | |||
| @if [ ! -f "$(GOPATH)/bin/golangci-lint" ] && [ ! -f "$(shell go env GOROOT)/bin/golangci-lint" ]; then \ | |||
| $(GO) install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.59.1; \ | |||
| $(GO) install github.com/golangci/golangci-lint/cmd/golangci-lint@latest; \ | |||
There was a problem hiding this comment.
| $(GO) install github.com/golangci/golangci-lint/cmd/golangci-lint@latest; \ | |
| $(GO) install github.com/golangci/golangci-lint/cmd/golangci-lint@1.60.2; \ |
golangci-lint의 버전을 명시하여 관리하는 편이 좋겠다는 의견을 말씀드립니다.
golangci-lint Changelog 로 볼때, v1.60.2를 적용하면, Go 1.23.0에 대한 lint를 지원할 수 있겠네요.
이후 커뮤니티 차원의 Go 버전을 업그레이드시에 알맞은 golangci-lint 버전을 함께 인하여 업그레이드 해 나가면 될 것 같고요.
개인적으로는 golangci-lint에서 지원하지 않는 Go 버전으로의 업그레이드는 재고해볼 필요가 있겠다는 인사이트를 얻을 수 있었습니다.
(참고) Changelog - https://golangci-lint.run/product/changelog/
Changelog
Follow the news and releases on Mastodon and on Twitter.
v1.60.2
- Updated linters
(중략)
- misc.
🎉 go1.23 support
yunkon-kim
changed the title
golangci-lint version supporting Go 1.23
|
PR 타이틀을 적당히 수정했습니다 ^^ |
|
@yunkon-kim @seokho-son |
Reverts #110
We can now revert this change to use the new linter after 03d77e7.