-
Notifications
You must be signed in to change notification settings - Fork 153
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
feat: upgrade to go 1.21 #317
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Chris Randles <randles.chris@gmail.com>
I was just trying to verify if this would break older go versions, and wrote this bash script and it seems to work. But I'm not a 100% sure. Wanted to understand if this is going to be a breaking change for those using v1.18-v1.20.
|
Perhaps worth adding back into the CI, then? I think things will break if you start depending on capabilities that don't exist yet in the older versions. Starting in 1.21, there's more flexibility. |
Yeah, it might be better to keep v1.18 -> v1.21 in the matrix. Having a similar discussion in #313. Let me think of other ways to test if anything is breaking in the meanwhile. |
Update all packages to 1.21, and update tests to cover 1.21 -> 1.23.
I was going to start with 1.23 -- I can update if preferred. At least with 1.21, you can build the later toolchains, so you can start to update individual packages as needed.
Additional changes:
There are many packages in the repository -- I'm not sure what your build tool of choice is, but here are some non-fancy makefile targets I've been using: