Skip to content

Conversation

@tani
Copy link

@tani tani commented Jan 11, 2019

Hello,

Since some jargon cannot be recognized by ginger, I added this options to skip errors.
This feature has already been implemented textlint-rule-spellchecker.
For me, it is enough to add this property from textlint-rule-spellchecker, however someone may want the other features. Since I have no time to do it, I stop to add the other features.

Anyway, I guess this request make your project to be better.

Thank you!

@tani
Copy link
Author

tani commented Jan 17, 2019

Hello @azu
Can you see this request? Please approve if you think no problem.

Copy link
Member

@azu azu left a comment

Choose a reason for hiding this comment

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

Thanks for Pull Request.

I recommend that you use @textlint/regexp-string-matcher in this usecase.

Or, introduce separated skipWords and skipRegExp option.

@tani
Copy link
Author

tani commented Jan 17, 2019

@azu I change it to skipPatterns, and added test for this change.

Copy link
Member

@azu azu left a comment

Choose a reason for hiding this comment

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

LGTM

@azu azu changed the title Add skipRegExps feat: Add skipPatterns option Jan 17, 2019
@azu azu changed the title feat: Add skipPatterns option feat: add skipPatterns option Jan 17, 2019
@azu azu merged commit 6b6ffca into textlint-rule:master Jan 17, 2019
@tani
Copy link
Author

tani commented Jan 17, 2019

Thank you! Could you bump up the version number and upload to npm?

@azu
Copy link
Member

azu commented Jan 17, 2019

Yes. I'll update devDeps before publishing.
It will take a few minutes.

@azu
Copy link
Member

azu commented Jan 17, 2019

@tani
Copy link
Author

tani commented Jan 17, 2019

Great!

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