-
Notifications
You must be signed in to change notification settings - Fork 6
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
deps: Update all backend dependencies #50
Conversation
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.
First of all, thanks a ton for all the work you made to put up those PRs. I will look into them as soon as I can. They all look great at the first glance. I have some suggestions so that we can discuss them on each PR.
Regarding the curren PR, I have very few minor comments. I think we need to update Validate plugin step as well as it uses the 1.21.x
Go as of now.
Signed-off-by: Jan-Otto Kröpke <joe@cloudeteer.de>
Thanks for the changes @jkroepke
We are still missing this one. Could you please update it? |
Signed-off-by: Jan-Otto Kröpke <joe@cloudeteer.de>
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.
Thanks a lot for the changes. LGTM!!
Now, the plugin validator breaks... But it works with the old version. Can be put the update of the plugin validator to a separat PR? |
Signed-off-by: Jan-Otto Kröpke <joe@cloudeteer.de>
Before looking into #41, I would like to start with the lastest dependencies.