-
Notifications
You must be signed in to change notification settings - Fork 671
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 to go 1.21 #4534
Update to go 1.21 #4534
Conversation
Signed-off-by: Eduardo Apolinario <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4534 +/- ##
==========================================
- Coverage 58.98% 58.93% -0.06%
==========================================
Files 621 620 -1
Lines 52483 52432 -51
==========================================
- Hits 30957 30900 -57
- Misses 19059 19069 +10
+ Partials 2467 2463 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
we need to update the go version in all the github checks too right? (ex flyte/.github/workflows/checks.yml Line 13 in aff1150
|
Signed-off-by: Eduardo Apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
* Update to go 1.21 Signed-off-by: Eduardo Apolinario <[email protected]> * Update go version in dockerfiles Signed-off-by: Eduardo Apolinario <[email protected]> * Use alpine 3.18 in component images Signed-off-by: Eduardo Apolinario <[email protected]> * Update other references to go 1.19 Signed-off-by: Eduardo Apolinario <[email protected]> --------- Signed-off-by: Eduardo Apolinario <[email protected]> Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]> Signed-off-by: Paul Dittamo <[email protected]>
Why are the changes needed?
What changes were proposed in this pull request?
How was this patch tested?
Setup process
Screenshots
Check all the applicable boxes
Related PRs
Docs link