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 golang to 1.23 #3395

Closed
2 tasks done
danail-branekov opened this issue Jul 18, 2024 · 0 comments
Closed
2 tasks done

Bump golang to 1.23 #3395

danail-branekov opened this issue Jul 18, 2024 · 0 comments

Comments

@danail-branekov
Copy link
Member

danail-branekov commented Jul 18, 2024

Although golang 1.23 has not been oficially released yet (ETA August 2024), it has been bumped in the Korifi container images by dependabot and all our tests are fine. Therefore we should probably look into doing that for our code as well.

Unfortunately, staticcheck fails with golang 1.23, see dominikh/go-tools#1578. Once this is resolved with staticcheck, we could go ahead and bump golang (see this wip branch)
In order to make staticcheck work, I had to pin it to 2024.1rc1 in the Makefile. latest staticcheck refers to the official release, see dominikh/go-tools#1578 (comment)

With that fixed, golangci-lint fails. It seems that it does not support 1.23 and will start supporting it when oficially released.

Once golangci-lint issue is resolved:

@danail-branekov danail-branekov converted this from a draft issue Jul 18, 2024
danail-branekov added a commit to eirini-forks/eirini-station that referenced this issue Jul 18, 2024
golangci-lint does not support go 1.23 yet, therefore we should not be
switching to it yet.

See cloudfoundry/korifi#3395

This reverts commit 345019c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

2 participants