-
-
Notifications
You must be signed in to change notification settings - Fork 919
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
fix(locale): filter inappropriate words in the en locale #1745
fix(locale): filter inappropriate words in the en locale #1745
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## next #1745 +/- ##
==========================================
- Coverage 99.63% 99.63% -0.01%
==========================================
Files 2337 2337
Lines 242692 242582 -110
Branches 1098 1098
==========================================
- Hits 241808 241698 -110
Misses 863 863
Partials 21 21
|
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.
I'm quite unsure about the filter and being a non-native speaker doesn't really help either.
Some words gets filtered, some other don't and I have trouble understanding why.
E.g. lust
got removed but lustrous
didn't.
Thanks for your time for checking the word lists. |
Lust means strong sexual desire Lustrous means shiny Different etymologies! I am a native English speaker and I can generally understand why words were removed. |
sometimes the words have both innocent and slang meanings, so a |
agree with your comments about the variants, i'll go through and have a second pass later. |
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.
Looks good to me, but the tests are failing.
Looks like CI runs into errors |
Following on from #1633
I ran the remaining words against https://github.com/zacanger/profane-words and found quite a few more words to remove.
Several of these are borderline, but the existing lists are so long I think there's little disadvantage in removing all of them.