-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Minor edits to the documentation #2140
Conversation
Nice! I almost couldnt believe we still had that many errors. Have a look at my comments and I’ll merge. |
Well, as a user, I found the documentation to be clear and comprehensive :)
I have made the requested changes but I wasn't entirely sure about the first one and so didn't resolve that conversation. |
Done :) |
Thanks!! |
Hey there,
First, thanks for your nice project :).
While reading the documentation, I found a few typos or missing punctuation, and thought you might want to receive a pull request for these.
Cheers,
Xavier
PS: I read the contributing.md, but it does not seem to apply to changes to the documentation?