Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Gender neutral language in the docs #745

Merged
merged 1 commit into from
Feb 2, 2021
Merged

Conversation

cpojer
Copy link
Contributor

@cpojer cpojer commented Feb 2, 2021

In the docs, I noticed there is an assumption in the example about the assistant's gender. This diff changes the text.

I had a version using "they" but when reading that back I was worried that non-native English speakers may get confused in this particular example since it is very early on in the docs and the example itself only serves as an elaboration of how draft state works. The version in this PR has the most clarity in my opinion, but I know that can be subjective. Happy to change it!

@mweststrate
Copy link
Collaborator

I agree, in this case the works as well and is simpler. Thanks!

@mweststrate mweststrate merged commit 02dd265 into immerjs:master Feb 2, 2021
@mweststrate
Copy link
Collaborator

🎉 This PR is included in version 8.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants