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

Add support for all alex config options. #7

Merged
merged 1 commit into from
Jul 31, 2020

Conversation

robdodson
Copy link
Contributor

  • Adds support for the other alex config options (deny, noBinary, profanitySureness).
  • Updates the README to include these options.

@robdodson
Copy link
Contributor Author

I'm not sure how the testing setup works. Perhaps it would save time for the repo owner to write the tests as they're more familiar with it?

@azu
Copy link
Member

azu commented Jul 31, 2020

@robdodson Thanks!

I'm not sure how the testing setup works. Perhaps it would save time for the repo owner to write the tests as they're more familiar with it?

OK.

If you can, please try to add a test case in another PR 👍

It is reasonable that we add deny option to valid pattern:
It test that deny option work.

        {
            text: "The boogeyman",
            options: {
                allow: ["boogeyman-boogeywoman"]
            }
        },
        {
            text: "The boogeyman",
            options: {
                // deny only "he-she" and "boogeyman" should be pass
                deny: ["he-she"]
            }
        }

{
text: "The boogeyman",
options: {
allow: ["boogeyman-boogeywoman"]
}
}

ℹ️ textlint-tester

@azu
Copy link
Member

azu commented Jul 31, 2020

I've released v2.1.0.

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