sapegin / textlint-rule-terminology

Textlint rule to check correct terms spelling
MIT License
50 stars 17 forks source link

Is it possible to exclude some terms from the config? #8

Closed pixelastic closed 5 years ago

pixelastic commented 6 years ago

Hello,

I've been trying to exclude some default transformation from the default list (namely, I don't want to transform websites into sites). I tried to achieve this goal by tweaking the terms key, but manage to have any impact.

Another viable solution might be to publish my own list on npm as you suggest on the README, but I could not find any definition of the format of this list. Should it be a simple JSON file?

sapegin commented 6 years ago

Yeah, terms is appending words to the default list, so you can't really remove anything.

Yeah, the format is the same as the default one.

Fee free to send a pull request with some ignore option, it may be simpler than publishing and maintaining your own list.

pixelastic commented 6 years ago

Thanks for the input. I've settled on storing my own terms file in my repo for the time being.

I did published my own list, but could not manage to import it. Would you have an example of a published external term list and how I could use it?

sapegin commented 6 years ago

I'm looking at the code, and I think it should be require.resolve instead of path.resolve:

https://github.com/sapegin/textlint-rule-terminology/blob/7fe73e048205904785735c03c471e21bcdebd72c/index.js#L73

pixelastic commented 5 years ago

I won't have the bandwidth to test and fix this at the moment unfortunately. I ended up using my own JSON list of terms in the repo, which works perfectly and is easier to maintain for us than an external list.

sapegin commented 4 years ago

:tada: This issue has been resolved in version 2.0.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: