-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update README.md to make contribution steps easier to follow #42
Conversation
Signed-off-by: Timothy Johnson <[email protected]>
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.
Should we make this a list of commands?
😋
LGTM!
Signed-off-by: Timothy Johnson <[email protected]>
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: Timothy Johnson <[email protected]>
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 pretty good.
A few things that you should address:
- Decide whether to use "plug-in" or "plugin." I see both styles in this content, and there should only be one.
- Decide whether to capitalize Web Help or not. I see both styles in this content, and there should only be one.
- Decide whether to italicize your steps. There's no ital in Installing, but there is ital in the other sections. There should only be one formatting style consistently used through this ReadMe.
Signed-off-by: Timothy Johnson <[email protected]>
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.
Nice job!
Update the readme to add installation steps and summary of each step. Thanks @robertblum-psi for suggesting improvements to the readme.