-
Notifications
You must be signed in to change notification settings - Fork 113
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
chore(dep): bumps go version to 1.19 #681
Conversation
- update go.mod / go.sum - update vendor directory - update build base image with go 1.19.3 Signed-off-by: Akhil Mohan <[email protected]>
Signed-off-by: Akhil Mohan <[email protected]>
Codecov Report
@@ Coverage Diff @@
## develop #681 +/- ##
===========================================
+ Coverage 45.01% 45.19% +0.17%
===========================================
Files 79 79
Lines 3992 3992
===========================================
+ Hits 1797 1804 +7
+ Misses 2028 2023 -5
+ Partials 167 165 -2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Signed-off-by: Akhil Mohan <[email protected]>
CI changes in PR #682. |
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 might want to remove the vendor directory.
@niladrih Arent we not keeping the vendor directory upto date now? I think with latest go version that should also work |
@akhilerm -- The vendor directory was required prior to go modules. We should be able to build without it... |
The changes are included in #682 |
Signed-off-by: Akhil Mohan [email protected]
Pull Request template
Why is this PR required? What issue does it fix?:
build was using a deprecated version of golang
What this PR does?:
Does this PR require any upgrade changes?: No
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:
<type>(<scope>): <subject>