-
Notifications
You must be signed in to change notification settings - Fork 124
Feature/mpdo 1015 #51
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
Conversation
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.
My suggestions are not required changes, but would improve the formatting and accessibility of the README.md
file.
It's good practice to specify some type of syntax highlighting for all fenced code blocks. If there is no specific syntax highlighting style for a particular block, just use
Anything that should be run on the command-line should use
If you want to show the output from a command, then prefix the command with a
|
@Djelibeybi - could you please approve and merge this PR, if everything looks good to you? |
Unfortunately, I can't approve or merge anything in this repo. I only reviewed it because I knew about it and thought the feedback might be useful. |
@Djelibeybi - could you pls let me know who can help in approving this PR? It's been pending for a long time now. |
@karankap I have no elevated access to any of the repos in this organisation, so I can't see any more than you can. |
Signed-off-by: Karan Kapoor karan.kapoor@oracle.com