Skip to content

Added all of the other languages from LDNOOBW - #1

Open
0x07dc wants to merge 1 commit into
roshanrahman:masterfrom
0x07dc:patch-1
Open

Added all of the other languages from LDNOOBW#1
0x07dc wants to merge 1 commit into
roshanrahman:masterfrom
0x07dc:patch-1

Conversation

@0x07dc

@0x07dc 0x07dc commented Feb 22, 2020

Copy link
Copy Markdown

Not sure if the english word list is the same as before. It was the most recent one on the LDNOOBW github

Not sure if the english word list is the same as before. It was the most recent one on the LDNOOBW github
@roshanrahman

Copy link
Copy Markdown
Owner

That's awesome. Adding more languages will definitely help, so let's implement it by letting the user pick which languages they want to add to the list (English will be default, others can be optionally enabled). I've taken your commit and added it to the version at pr/1 branch of this repo.

@apalala-dev

Copy link
Copy Markdown

I am considering using this plugin but I would like to use if for non-english words aswell.
Is there a reason why this PR has not been merged into main branch?

@roshanrahman

Copy link
Copy Markdown
Owner

Hi, while it will be helpful to have lists for other languages, the English list was originally intended to simply act as a default list (on which a custom list of blacklisted words could be made). I'll see if I can add the other languages as something the user can optionally import, whenever I have the time.

In the meantime, you can use the filterAdditionally() constructor and pass in a custom list of words that you want to check against, along with the English list.

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