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

Go 1.22 -> 1.23 업데이트합니다. #99

Merged
merged 2 commits into from
Nov 26, 2024
Merged

Go 1.22 -> 1.23 업데이트합니다. #99

merged 2 commits into from
Nov 26, 2024

Conversation

litsynp
Copy link
Contributor

@litsynp litsynp commented Nov 24, 2024

golangci-lint에서 너무 많은 램 사용량(16GB+)을 보여서 봤더니 go와 golangci-lint의 go 버전 불일치로 인한 이슈였습니다.

Go 버전을 업데이트합니다.

@litsynp litsynp added the build Related to build scripts, CI/CD, and dependencies label Nov 24, 2024
@litsynp litsynp requested a review from JoeCP17 November 24, 2024 14:30
@litsynp litsynp self-assigned this Nov 24, 2024
@JoeCP17
Copy link
Contributor

JoeCP17 commented Nov 24, 2024

제가 GO 언어에 대한 고찰이 아직 부족해서 그럴수도 있다고 생각들긴하는데, 릴리즈 속도가 꽤나 빠른거같아보이는데,
언제 시간될때 버전 정착해서 사용하는건 어떤가요 latest로 설정되어있으니까 이렇게 바꿔야하는 케이스가 너무많은거 같아보여서요

@litsynp
Copy link
Contributor Author

litsynp commented Nov 25, 2024

제가 GO 언어에 대한 고찰이 아직 부족해서 그럴수도 있다고 생각들긴하는데, 릴리즈 속도가 꽤나 빠른거같아보이는데, 언제 시간될때 버전 정착해서 사용하는건 어떤가요 latest로 설정되어있으니까 이렇게 바꿔야하는 케이스가 너무많은거 같아보여서요

@JoeCP17

Go가 변경에 대해 굉장히 보수적이고 버전별로 하위 호환에 신경을 많이 쓰기 때문에 버전은 항상 최신 버전으로 올려도 된다고 판단했습니다~
레포에서 버전이 latest로 설정되어 있진 않고, 고정시켜두고 신 버전 나오면 그거보다 조금 뒤에 올리고 있어요.

Memory issue는 Go 자체보단 툴의 문제고, 제가 nix를 쓰기 때문에 아마 의빈님 컴퓨터에선 발생 안했을 것 같긴 해요.

golangci-lint 같은 경우엔 주기적으로 업데이트하는 걸 권장하고 있습니다.

It is advised that you periodically update the version of golangci-lint as the project is under active development and is constantly being improved. For any problems with golangci-lint, check out recent GitHub issues and update if needed. (출처)

Go 버전을 올린 건 겸사겸사라고 봐주시면 될 것 같습니다.

@JoeCP17
Copy link
Contributor

JoeCP17 commented Nov 26, 2024

@litsynp
아하 lint 공식문서까지는 제가 디테일하게 보질 않았어서, 잚 몰랐네요
확인했습니다~

@litsynp litsynp merged commit 029186b into main Nov 26, 2024
1 check passed
@litsynp litsynp deleted the build/go-1.23 branch November 26, 2024 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to build scripts, CI/CD, and dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants