-
Notifications
You must be signed in to change notification settings - Fork 107
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
build(go): switch to go 1.21 #2049
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2049 +/- ##
==========================================
- Coverage 92.92% 92.89% -0.03%
==========================================
Files 165 165
Lines 21582 21582
==========================================
- Hits 20054 20048 -6
- Misses 952 956 +4
- Partials 576 578 +2 ☔ View full report in Codecov by Sentry. |
18bc348
to
65ce943
Compare
ea08507
to
5420ce9
Compare
2cf93d2
to
b1a0cd3
Compare
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
The ci failure is expected since we are not making it aware that go version has changed across this PR. |
@andaaron pls sign the commit and re-push. |
Also update to the latest swaggo version, as the previous one did not work with go 1.21 Signed-off-by: Andrei Aaron <[email protected]>
Also update to the latest swaggo version, as the previous one did not work with go 1.21
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.