-
Notifications
You must be signed in to change notification settings - Fork 55
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
Updating the README.md #38
Conversation
Updating the README.md to include information on prerequisites, installation, and feature set.
Updating the README.md
Hi @Tatsinnit and @itowlson, |
💡Cool, Thank you for the ping. Couple of observations + minor note: 📓 I was thinking that there will be an Observation:
|
Incorporating feedback from Tats :)
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.
This looks good - so nice to have someone putting in the documentation effort to give it that "finished product" look! Thank you!
I've suggested a few changes but they're pretty nitpicky - feel free to skip over them if time is short or if you disagree.
Switching back to the bullet form + removing prerequisites section
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 good!
Adding |
…-screenshots Update yunjung fork reamdme with screenshots
Thanks, cc: @itowlson + @yunjchoi - looks good to me now, please eye ball if there is anything else left otherwise from my end this looks
|
🎉 |
Updating the README.md to include information on prerequisites, installation, and feature set.