-
Notifications
You must be signed in to change notification settings - Fork 88
Minor updates to docs #837
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
@dimitri If you could review the small docs changes here, we can pull them for 0.17 perhaps |
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 for the docs review! Many good changes, a lot of stylistics changes that I don't know if they are improvements, because I'm not a native English speaker, but I would default to accept them.
That said, there are a small number of misinformed/misleading changes in your PR that I think should be fixed before merge. Please consider fixing them as per the detailed review below.
9288110
to
e8e2df5
Compare
Commented and sent the new changes commit |
Some edits regarding fixing typos and expressions for better readability and grammar. Note that Reference Manual section is omitted and shall not be included.