-
Notifications
You must be signed in to change notification settings - Fork 102
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
feat(deps): Bump golang, k8s and lib-csi versions #444
Conversation
7757ae0
to
66d574d
Compare
f71facd
to
de27920
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.
Hey, thanks for this change. Might be a good idea to remove the vendor directory.
This entails few other minor changes:
- Github actions workflows exclusions
- Makefile
-mod=vendor
to-mod=mod
options. Removing the module paths' 'vendor/' prefix. - makefile grep command exclusions
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.
Might be a good idea to switch out github.com/ghodss/yaml with sigs.k8s.io/yaml@v1.3.0
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.
Might be a good idea to switch out github.com/ghodss/yaml with sigs.k8s.io/yaml@v1.3.0
- go: 1.19.9 - k8s: v1.27.2 - klog: v2 - lib-csi: v0.7.0 Signed-off-by: shubham <shubham14bajpai@gmail.com>
What this PR does?:
This PR updates the following components:
Does this PR require any upgrade changes?:
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>