Skip to content

Conversation

@gauravbisht005
Copy link
Contributor

Hope this is what you wanted!!
Else let me know... I'll ammend.

@csviri
Copy link
Collaborator

csviri commented Oct 22, 2020

Hi @gauravbisht005 thank you for PR.

One remark, do we really need html for this?
See second answer here: https://stackoverflow.com/questions/2822089/how-to-link-to-part-of-the-same-document-in-markdown

would be nice to have it as pure markdown

@gauravbisht005
Copy link
Contributor Author

Hi @gauravbisht005 thank you for PR.

One remark, do we really need html for this?
See second answer here: https://stackoverflow.com/questions/2822089/how-to-link-to-part-of-the-same-document-in-markdown

would be nice to have it as pure markdown

Okay!! Thank you for the feedback. I'll make the changes by the evening(IST 2000Hrs)

@gauravbisht005
Copy link
Contributor Author

I've made the amendments as stated!
Thank you :)

Copy link
Collaborator

@csviri csviri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, yep great thx!!

@csviri csviri merged commit a98e88c into operator-framework:master Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants