-
-
Notifications
You must be signed in to change notification settings - Fork 720
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
pkg/CI: Update go version directive in all modules to 1.20, update CI spec to support last 2 major releases #796
Conversation
Squashed commit of the following: commit de18fe9 Author: Ken Hibino <[email protected]> Date: Sun Sep 17 19:35:33 2023 -0700 Update README about supported Go versions commit 714d62b Author: Ken Hibino <[email protected]> Date: Sun Sep 17 19:26:16 2023 -0700 Bound build to the latest two go versions
Squashed commit of the following: commit 6e3656db222a3f9347ee4806ef065a1b9b01a214 Author: Mohammed Sohail <[email protected]> Date: Thu Dec 7 11:12:41 2023 +0300 pkg(x): go version update -> 1.20 commit 2931df3 Author: Amaury <[email protected]> Date: Wed Dec 6 17:47:03 2023 +0100 fix tests commit 1122780 Author: Amaury <[email protected]> Date: Wed Dec 6 16:40:32 2023 +0100 chore: Update redis to v9 in x/go.mod
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #796 +/- ##
=======================================
Coverage 68.37% 68.37%
=======================================
Files 27 27
Lines 3836 3836
=======================================
Hits 2623 2623
Misses 929 929
Partials 284 284 ☔ View full report in Codecov by Sentry. |
* closes #759 Squashed commit of the following: commit 3d94ee14aeaf9a868dbeed4b65f90ccdda1f08d6 Author: Mohammed Sohail <[email protected]> Date: Thu Dec 7 11:49:07 2023 +0300 ci: upgrade benchstat actions, go version -> 1.21.x commit 129e225 Author: angshumukherjee100 <[email protected]> Date: Sun Oct 8 11:20:43 2023 +0530 (workflow): bump go version to 1.18 in benchstat
@kamikazechaser do you have merge rights? Or are we waiting on @hibiken? |
I do have it. But We have agreed on 2/2 for not-critical PRs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @kamikazechaser for working on this!
Closes #719. See the discussion for more information.
Also aims to support new language features added after 1.14. E.g. #677.
Other info
x
module dependencies.Test changes