-
Notifications
You must be signed in to change notification settings - Fork 607
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 runner to Go 1.22,1.23 (two most recent releases). #887
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #887 +/- ##
==========================================
+ Coverage 66.86% 67.00% +0.13%
==========================================
Files 44 44
Lines 9824 9798 -26
==========================================
- Hits 6569 6565 -4
+ Misses 2794 2780 -14
+ Partials 461 453 -8 ☔ View full report in Codecov by Sentry. |
v1.51 fails with errors related to old slices package.
with: | ||
# Bootstrapping go tip requires 1.22.6 | ||
go-version: 1.22 | ||
cache: true |
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.
Why are these cache manipulation needed?
May worth a comment.
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.
Without these we get errors from setup-go about it not being able to use the cache. (I think it might be related to having multiple go.sum files in our repo). I will add a comment.
This PR should now be mergeable. |
No description provided.