-
-
Notifications
You must be signed in to change notification settings - Fork 614
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
Update go versions in CI and release #7971
base: main
Are you sure you want to change the base?
Conversation
Three classes of errors currently:
|
I've fixed the |
I've fixed all the remaining code errors -- all that's left is the golangci-lint failure. |
a886006
to
c2379fe
Compare
The new go1.24 FIPS-compliant crypto/rsa package refuses to work with the integration test's keys with too-close-together factors. We can still unit-test the underlying function, because that operates on big ints rather than keys, but the integration test now errors during setup.
c2379fe
to
709ae64
Compare
Updated to go1.23.6 and go1.24.0, now that it's been officially released. Updated golangci-lint to 1.64.0, which supports go1.24.0. Updated staticcheck to 2025.1, which supports go1.24.0. Fixed a lot of lint complaints, and added a new linter which will help us more narrowly target and document our lint exceptions. Now just waiting on the official go1.24.0 docker image for the release workflows. |
Update from go1.23.1 to go1.23.6 for our primary CI and release builds. This brings in a few security fixes that aren't directly relevant to us.
Add go1.24.0 to our matrix of CI and release versions, to prepare for switching to this next major version in prod.