textlint-ja / textlint-rule-no-doubled-joshi

文中に同じ助詞が複数出てくるのをチェックするtextlintルール
MIT License
22 stars 4 forks source link

feat(option): add new option "allow" #11

Closed takahashim closed 8 years ago

takahashim commented 8 years ago

先ほどはmergeありがとうございました。 こちらが本題で、「AやBや」「AもBも」等を許すためにallowed_joshiというオプションを.textlintrcに追加するものです。 allowed_joshiという名前はあまり分かりやすくないような気もするので、なにか分かりやすい命名があれば変更した方がよいかもしれません(ignored / excluded / skip 等…?)

azu commented 8 years ago

基本的にはESLintのやり方を真似るのが分かりやすいかなと思います。 確か同じような例外オプションがあるルールがあった気がするので調べてみます。 後はキャメルケースの方が見慣れる感じはありますね。

azu commented 8 years ago

ESLintもばらけているとメンテナーの人に教えてもらいました…

が基本的には "allow": 配列 というパターンが多いようです。

http://eslint.org/docs/rules/no-empty-function http://eslint.org/docs/rules/no-console

(今のところ)ユーザーが手入力するオプション名なので、できるだけ簡潔にした方がtypoとかが減っていいと思うので、joshi というわかりにくい名前も外してallowだけでいいと思います。

"allow": []

助詞以外にもallowするパターンが出てきてしまったら破綻しますが、その場合はルールが単一責任以外のことをやっているという裏返しになっていると思います

takahashim commented 8 years ago

なるほどです。allowにしてみました。 squashした方がよいですか?

azu commented 8 years ago

merge時にsquash出来るので気にしなくて大丈夫ですー

azu commented 8 years ago

3.3.0としてリリースしました。

takahashim commented 8 years ago

ありがとうございます!