-
Notifications
You must be signed in to change notification settings - Fork 66
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
Switch the bundle validate to use the API #770
Switch the bundle validate to use the API #770
Conversation
Waiting on: operator-framework/api#255 |
ad6b03e
to
81b90a3
Compare
81b90a3
to
808e8f8
Compare
808e8f8
to
699f5b2
Compare
052d0d0
to
4338934
Compare
4338934
to
ffec18f
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.
Looks good for the most part. I think part of the challenge I have with this particular collection of checks is that the logic is feels somewhat tricky to follow. That's neither here nor there for this PR, though.
ffec18f
to
8f94d5e
Compare
This uses the validators directly, but still has the same usage semantics, for the most part. Signed-off-by: Brad P. Crochet <brad@redhat.com>
8f94d5e
to
3ce058c
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: acornett21, bcrochet, komish The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This uses the validators directly, but still has the same usage
semantics, for the most part.
Signed-off-by: Brad P. Crochet brad@redhat.com