get-alex / alex

Catch insensitive, inconsiderate writing
https://alexjs.com
MIT License
4.8k stars 207 forks source link

Adding reminder for inclusive pronouns Issue #297 #345

Open jfreud1 opened 9 months ago

jfreud1 commented 9 months ago

Closes issue #297 This implementation is for the second solution proposed in the issue "More Helpful Response to He/Her Pronouns" Let us know what you think! Our changes were made to index.js this function is intended to remind the user to confirm the proper pronouns of the subjects in the text when pronouns are detected

wooorm commented 8 months ago

Thanks for your PR and your patience!

That issue is open because I have feedback on it that has not been addressed by the original poster. To recap: it sounds like the user knows perfectly well what to do. I am not sure this feature is needed. More here: https://github.com/get-alex/alex/issues/297#issuecomment-671773658.

I am not sure a message is needed for this. And I am not sure about randomly logging messages.