Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Added a couple more notes and modifications & modified the tests to accommodate them #49

Merged
merged 8 commits into from
Oct 12, 2018
Merged

Added a couple more notes and modifications & modified the tests to accommodate them #49

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
a5f0701
Added a whole lot of notes
AhmedRedaAmin Oct 5, 2018
829f414
modified some tests to accomodate changes
AhmedRedaAmin Oct 6, 2018
462b3a1
some tests were still failing -fixed
AhmedRedaAmin Oct 6, 2018
ca727c1
removed upper case & non-offensive words
AhmedRedaAmin Oct 7, 2018
5055b34
fixed merge conflicts
AhmedRedaAmin Oct 7, 2018
085469d
Merge branch 'master' into master
AhmedRedaAmin Oct 8, 2018
e597a47
Merge branch 'master' into master
AhmedRedaAmin Oct 12, 2018
98b64b8
Update test.js
AhmedRedaAmin Oct 12, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions script/ablist.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
- afflicted with a disability
- type: simple
source: http://ncdj.org/style-guide/
note: People with special abilities is a gentler term that conveys the same meaning.
considerate:
- people with intellectual disabilities
inconsiderate:
Expand All @@ -39,6 +40,7 @@
- person with an intellectual disability
inconsiderate:
- intellectually disabled
- has intellectual issues
- suffers from intellectual disabilities
- suffering from intellectual disabilities
- suffering from an intellectual disability
Expand All @@ -48,6 +50,7 @@
source: http://ncdj.org/style-guide/
considerate:
- rude
- malicious
- mean
- disgusting
- vile
Expand Down Expand Up @@ -218,19 +221,22 @@
- person recovering from a drug addiction
inconsiderate:
- addict
- junkie
- type: simple
source: http://ncdj.org/style-guide/
considerate:
- people with a drug addiction
- people recovering from a drug addiction
inconsiderate:
- addicts
- junkies
- type: simple
source: http://ncdj.org/style-guide/
considerate:
- someone with an alcohol problem
inconsiderate:
- alcoholic
- alcohol abuser
- type: simple
source: http://ncdj.org/style-guide/
considerate:
Expand Down Expand Up @@ -269,6 +275,8 @@
inconsiderate:
- vertically challenged
- midget
- small person
- dwarf
- type: simple
source: http://ncdj.org/style-guide/
considerate:
Expand Down Expand Up @@ -406,6 +414,7 @@
- sociopaths
- type: simple
source: http://www.autistichoya.com/p/ableist-words-and-terms-to-avoid.html
note: Dumb here is used in 2 different contexts , the inability to talk or as a curse word.
considerate:
- foolish
- ludicrous
Expand All @@ -424,6 +433,7 @@
- stupid
- wacko
- whacko
- low iq
- type: simple
considerate:
- fit of terror
Expand All @@ -436,14 +446,19 @@
- slim
inconsiderate:
- anorexic
- bony
- skinny
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, what was the reason to add skinny here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well , good question , the thing is , skinny comes from a description that someone is "skin on bones" which is used to reference people in late stages of starvation , though of course recently it has been normalized in day to day English as a casual word , however, if we are to level this description at someone who actually has anorexia for example , I thought it sounded inconsiderate in that context , or you know , I could just be overthinking this :'D.
But yeah , here is how my thought process went for both "bony" and "skinny" .
Anyways , I'll be removing skinny now .

Copy link
Contributor Author

@AhmedRedaAmin AhmedRedaAmin Oct 7, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also concerning the upper case in race.yml , I just capitalized the races ,regions , nationalities and whatnot , I wasn't aware the tooling puts everything to lower case(or uppercase) before matching , so I 'll be on it 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Finally , it's my pleasure , mate ! 👍

- type: simple
source: http://english.stackexchange.com/questions/247550/
note: If possible , specify which anxiety disorder is intended and its restrictions.
considerate:
- has an anxiety disorder
- obsessive
- pedantic
- niggly
- picky
inconsiderate:
- neurotic
- ocd
- o.c.d
- o.c.d.
Expand Down Expand Up @@ -532,3 +547,10 @@
- enthusiast
inconsiderate:
- maniac
- type: simple
considerate:
- person with prominent teeth
- prominent teeth
inconsiderate:
- bucktoothed
- buckteeth
58 changes: 58 additions & 0 deletions script/race.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,3 +50,61 @@
- urban
inconsiderate:
- ghetto
- type: simple
considerate:
- Native American
inconsiderate:
- red Indian
- Pocahontas
- type: simple
considerate:
- Native American People
inconsiderate:
- red Indians
- type: simple
considerate:
- Filipinos
- Filipino people
inconsiderate:
- pinoys
- pinays
- type: simple
considerate:
- Arabs
- Middle Eastern People
inconsiderate:
- sand niggers
- towel heads
- type: simple
note: Whenever possible , always try to be gender inclusive.
considerate:
- Latinx
inconsiderate:
- Latino
- Latina
- Mexican
- type: simple
considerate:
- Japaneses person
- Japanese people
inconsiderate:
- Japs
- type: simple
considerate:
- Jewish person
inconsiderate:
- kike
- goyum
- goy
- Shlomo
- type: simple
considerate:
- a Black person
inconsiderate:
- spade
- type: simple
considerate:
- Nomad
inconsiderate:
- gyppo
- gypsy
8 changes: 4 additions & 4 deletions test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ test('retext-equality', function(t) {
t.same(
process('Eric is mentally ill.'),
[
'1:9-1:17: `mentally ill` may be insensitive, use `rude`, `mean`, `disgusting`, `vile`, `person with symptoms of mental illness`, `person with mental illness`, `person with symptoms of a mental disorder`, `person with a mental disorder` instead'
'1:9-1:17: `mentally ill` may be insensitive, use `rude`, `malicious`, `mean`, `disgusting`, `vile`, `person with symptoms of mental illness`, `person with mental illness`, `person with symptoms of a mental disorder`, `person with a mental disorder` instead'
],
'ablist-language'
)
Expand Down Expand Up @@ -200,7 +200,7 @@ test('Phrasing', function(t) {
t.same(
process('This is insane.'),
[
'1:9-1:15: `insane` may be insensitive, use `rude`, `mean`, `disgusting`, `vile`, `person with symptoms of mental illness`, `person with mental illness`, `person with symptoms of a mental disorder`, `person with a mental disorder` instead'
'1:9-1:15: `insane` may be insensitive, use `rude`, `malicious`, `mean`, `disgusting`, `vile`, `person with symptoms of mental illness`, `person with mental illness`, `person with symptoms of a mental disorder`, `person with a mental disorder` instead'
],
'This is insane'
)
Expand Down Expand Up @@ -264,7 +264,7 @@ test('Phrasing', function(t) {
t.same(
process('Yeah, you were really psycho to him.'),
[
'1:23-1:29: `psycho` may be insensitive, use `rude`, `mean`, `disgusting`, `vile`, `person with symptoms of mental illness`, `person with mental illness`, `person with symptoms of a mental disorder`, `person with a mental disorder` instead',
'1:23-1:29: `psycho` may be insensitive, use `rude`, `malicious`, `mean`, `disgusting`, `vile`, `person with symptoms of mental illness`, `person with mental illness`, `person with symptoms of a mental disorder`, `person with a mental disorder` instead',
'1:33-1:36: `him` may be insensitive, use `their`, `theirs`, `them` instead'
],
'Yeah, you were really psycho to him'
Expand Down Expand Up @@ -311,7 +311,7 @@ test('Phrasing', function(t) {
t.same(
process('My O.C.D. is coming out again!'),
[
'1:4-1:10: `O.C.D.` may be insensitive, use `Obsessive`, `Pedantic`, `Niggly`, `Picky` instead'
'1:4-1:10: `O.C.D.` may be insensitive, use `Has an anxiety disorder`, `Obsessive`, `Pedantic`, `Niggly`, `Picky` instead'
],
'My O.C.D. is coming out again!'
)
Expand Down