-
Notifications
You must be signed in to change notification settings - Fork 5.9k
More grammar and typo fixes. #458
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
Also fixed a small typo.
Also added a brief description for PyQt.
Tried to normalize with no Oxford commas.
docs/dev/env.rst
Outdated
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.
Can we please avoid language that makes it harder for those reading the guide who are not native English speakers? enable, set, configure, anything like that would be 100x more obvious to a someone who's first language is not English.
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.
enable wasn't quite a correct usage. I'll change it to set, that's better.
|
many thanks! |
More grammar and typo fixes.
The rest of the changes started in PR #456.