-
Notifications
You must be signed in to change notification settings - Fork 24
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
go.mod: upgrade to newer versions #307
Conversation
Thanks! This has been on my TODO list for a while so I'm happy to see this happen. |
Commit message nitpick: change |
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.
A nitpick, nothing major
36cd62a
to
e81c49a
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. Never hesitate to ping me if I ask for something and then you don't hear back from me for a while. :-)
@Mergifyio rebase |
✅ Branch has been successfully rebased |
e81c49a
to
1ea49a8
Compare
🎉 All dependencies have been resolved ! |
@Mergifyio rebase |
Upgrade logr, client-go and controller-runtime to newer versions which include various bug fixes. Align code with respect to changes in those libraries. Signed-off-by: Shachar Sharon <ssharon@redhat.com>
✅ Branch has been successfully rebased |
1ea49a8
to
af1ebe3
Compare
Upgrade logr, client-go and controller-runtime to newer versions which include various bug fixes. Align code with respect to changes in those libraries.
depends on #310