-
Notifications
You must be signed in to change notification settings - Fork 82
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 quic to use go 1.22 #1214
Conversation
i think with this PR we can also change the github actions to move to 1.22, Or have the actions PR merged first, what do you think? |
google.golang.org/appengine v1.6.8 // indirect | ||
google.golang.org/protobuf v1.33.0 // indirect | ||
google.golang.org/protobuf v1.34.1 // indirect |
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.
you will also need to delete line 96 😄
Codecov ReportAll modified and coverable lines are covered by tests ✅
@@ Coverage Diff @@
## devel #1214 +/- ##
==========================================
+ Coverage 39.06% 39.71% +0.65%
==========================================
Files 45 48 +3
Lines 9260 9840 +580
==========================================
+ Hits 3617 3908 +291
- Misses 5371 5660 +289
Partials 272 272 see 5 files with indirect coverage changes
🚨 Try these New Features:
|
Quality Gate passedIssues Measures |
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.
lgtm
Update quic to use go 1.22