-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Edit security-getting-started-tutorial.adoc #37817
Conversation
🙈 The PR is closed and the preview is expired. |
Thanks @rolfedh LGTM, a couple of open questions but otherwise a nice update as always |
@rolfedh Are you awaiting for more reviews or is it ready to be merged ? |
@sberyozkin Hello there! I will review it soon and that we are good to go! |
@rolfedh @sberyozkin My review is done. Looks good! |
Thanks @rolfedh @MichalMaler, as it happens, in meantime, the PR I created earlier in response to the QE doc review has been merged, so Rolfe, please resolve the conflict |
Thank you for your review, @MichalMaler! |
References https://issues.redhat.com/browse/QDOCS-564