-
Notifications
You must be signed in to change notification settings - Fork 53
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
Clean-up of operator conditions with helper funcs #228
Clean-up of operator conditions with helper funcs #228
Conversation
Needs a rebase after merging #218. |
Signed-off-by: Joaquim Moreno Prusi <joaquim@redhat.com>
13f8591
to
665288a
Compare
fixed! |
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
@tmshort thanks! Can you please attach this one to the OLM v1 and v0.2.0 milestone? I can't do it myself. |
This is not an issue... |
Yeah, looks as though it need to be an issue to be viewed in the milestones. |
@tmshort You should be able to manually set the milestone on this PR to show up in the v0.2.0 release milestone. I don't think the OLMv1 GH project is configured to show PRs (I think they can be configured to show PRs) |
@everettraven right, I can set the label, but it won't show it under M4... but it looks as though it's showing up under v0.2.0...? |
(It would also help if I look just under the labels... D'Oh) |
Description
Clean-up of operator conditions with helper funcs.
Requires #218
follow-up of #218 (comment)
Reviewer Checklist