-
Notifications
You must be signed in to change notification settings - Fork 33
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
Release v0.5.0 #343
Release v0.5.0 #343
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #343 +/- ##
==========================================
+ Coverage 64.64% 65.03% +0.39%
==========================================
Files 35 35
Lines 3804 3804
==========================================
+ Hits 2459 2474 +15
+ Misses 1147 1134 -13
+ Partials 198 196 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
|
something to comment that I'll also add it to the RFC: The push event that triggers the build of the images are not passing all the params needed. I usually cancel it, then trigger manually the workflow_dispatch "Build Images" with the same params you ran the |
I reckon the manifest failed because of your local version of |
8a06bd7
to
781c23c
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.
Should we update https://github.com/Kuadrant/kuadrant-operator/blob/main/CHANGELOG.md as well?
Oops, @guicassolato we (royal) forgot to do it the last time, but yes, we should do it |
So, I will redo this yet again, but I challenge the usefulness of the changelog in this case... |
781c23c
to
6d706c5
Compare
Not a PR to merge, just checking this is correct and we can run the job:
see how this goes...