Use more inclusive verbiage #340

Closed
wants to merge 1 commit into
from

4 participants

@i-am-tom
i-am-tom commented May 1, 2016 edited

This is made in reference to point 1 of the code manifesto, really, and updates documentation accordingly.

This might sound like nitpicking, but I think we'd probably be better off avoiding these references to mental health; thoughts?

@i-am-tom i-am-tom Removing the use of 'sane'
RE: point 1 of the code manifesto, really. This might sound like nitpicking, but I think we'd probably be better off avoiding these references to mental health; thoughts?
f7d08d8
@grizzm0

The context is not regarding a person. In this case sane would mean something more like reasonable/sensible and has nothing to do with mental health.

@i-am-tom

I totally understand what the word means in this context, and my point is just that there are better ways of phrasing it that don't have any potential to be exclusionary :) This same reasoning is why we avoid words like "retarded" and "dumb" - the context might not have anything to do with humans, but they're needless slurs, and there are plenty of other ways to say the same thing.

@weierophinney
Zend Framework member

The wording we use is verbatim from the code manifesto; if you have suggestions on wording, please make them against its own repository, and we will pull them in if merged.

@akrabat
Zend Framework member
@weierophinney
Zend Framework member

This is what comes of me reading PRs on my phone... Re-opening, and will re-evaluate. Thanks for the PR, @i-am-tom !

@weierophinney weierophinney reopened this May 3, 2016
@weierophinney weierophinney changed the title from Removing the use of 'sane' to Use more inclusive verbiage May 3, 2016
@weierophinney weierophinney self-assigned this May 3, 2016
@weierophinney weierophinney added a commit that referenced this pull request May 3, 2016
@weierophinney weierophinney Merge branch 'hotfix/340' into develop
Forward port #340
98922c1
@weierophinney weierophinney added a commit that closed this pull request May 3, 2016
@weierophinney weierophinney Merge branch 'hotfix/340'
Close #340
29a6578
@weierophinney
Zend Framework member

I back-ported the changes to the documentation source files, which are under the doc/book/ tree of the master and develop branches, and written in markdown. Documentation rebuilds after pushes to master, so you'll see your change show up shortly.

Thanks for the patch, @i-am-tom !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment