-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix(README): overall improvements #2080
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
This is great thank you Jeff! I’m busy with other work right now but I’ll get back to your PR with a more thorough review soon |
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.
Some feedback and further suggestions, please let me know what you think. Thanks again for helping with the documentation, I really appreciate it, and I really need more help with documentation in general
That all looks good @gr2m. I updated the PR with your suggestions |
Add @gr2m's comments Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com>
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.
Thanks a lot Jeff!
The doc is looking good. I found a few typos and wording issues, and have a few suggestions. Summary:
View rendered README.md