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.

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

Added badWords list #1315

Closed
wants to merge 1 commit into from
Closed

Added badWords list #1315

wants to merge 1 commit into from

Conversation

Bossett
Copy link

@Bossett Bossett commented Jul 13, 2023

Note: Closed in favour of #1314 which already did some curation - the list here is TOO BROAD and I didn't continue fixing it as that work was done already in the other PR.

Pulled list from https://github.com/dariusk/wordfilter/blob/master/lib/badwords.json and added new badWords list for general bans, slurs, etc.

Obviously not the long-term solution - but hopefully something that can be quickly checked and implemented.

May be worth pluralising terms, extending matching, doing cleverness to match Unicode, etc. etc.; but this is the quick-and-dirty.

Source is https://github.com/dariusk/wordfilter/blob/master/lib/badwords.json from https://github.com/dariusk/wordfilter/

Pulled list from https://github.com/dariusk/wordfilter/blob/master/lib/badwords.json and added new badWords list for general bans, slurs, etc.
@azigler
Copy link

azigler commented Jul 13, 2023

Would address #1317

@jesslark
Copy link
Contributor

Would close #1317

I specifically removed some words. As a fat person I see no reason to block those handles. Some I removed might be something a SW uses. I'm fine with whatever PR gets merged to address, at the very least, the most common racial slurs. We can work the rest out as we move forward.

@Bossett
Copy link
Author

Bossett commented Jul 13, 2023

Closed in favour of @jesslark's PR 1314 which already did some curation.

@Bossett Bossett closed this Jul 13, 2023
@Bossett Bossett deleted the patch-1 branch July 13, 2023 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants