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 a profanity #1

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Added a profanity #1

wants to merge 2 commits into from

Conversation

randolf
Copy link

@randolf randolf commented Nov 22, 2024

Added a profanity that was introduced in a South Park (TV series) episode about curse words.

Added a profanity that was introduced in a South Park (TV series) episode about curse words.
Copy link
Owner

@dsojevic dsojevic left a comment

Choose a reason for hiding this comment

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

Hi @randolf, thanks for the contribution!

The .json files are the source of truth and build out the corresponding .txt files after running ./bin/build.js

If you're happy with your changes to en.json, you can run the build script and it will generate the desired en.txt file for it.

@@ -2477,6 +2477,10 @@
"id": "horseshit",
"match": "horseshit|horse-shit|horse shit"
},
{
"id": "meecrob",
"match": "pile of meecrob"
Copy link
Owner

Choose a reason for hiding this comment

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

This JSON takes pipe-separated values for the match attribute and will turn those into individual items in the en.txt list; the id attribute value isn't added to the .en.txt list from this though.

Based on your original change to en.txt, this would instead be:

Suggested change
"match": "pile of meecrob"
"match": "meecrob"

Otherwise, if you wanted to see the full match for either/or you could use:

Suggested change
"match": "pile of meecrob"
"match": "meecrob|pile of meecrob"

Though just defining meecrob for the match would probably result in the desired behaviour for most implementations using this list, e.g. piece of meecrob -> piece of *******

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.

2 participants