-
Notifications
You must be signed in to change notification settings - Fork 39
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
✨ Add ALM Examples for new CRs #369
Conversation
@jortel can you please have a quick look and make sure these examples aren't insane. |
@fbladilo heads up on this one too. In downstream won't take the ALM example templates into account and you need to manually update the CSV (or ignore this/me if the opposite is true.) |
706387d
to
716cc17
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
Signed-off-by: Jason Montleon <jmontleo@redhat.com>
716cc17
to
aa0e487
Compare
@jmontleon Thanks for heads up, yes we need to manually add these to downstream templates |
No description provided.