textlint-ja / textlint-rule-max-ten

textlint rule that limit maxinum ten(、) count of sentence.
MIT License
10 stars 5 forks source link

「,.」を指定できるようにしたい #7

Closed nrkt closed 3 years ago

nrkt commented 3 years ago

現状は「、。」のチェックのみとなっていますが、「,.」をチェックできるようになるととてもありがたいです!

azu commented 3 years ago

https://github.com/textlint-ja/textlint-rule-max-ten/blob/3dfb5f458d170079401c5a501553ce46c59eaa2b/src/max-ten.js#L51 https://github.com/textlint-ja/textlint-rule-max-ten/blob/3dfb5f458d170079401c5a501553ce46c59eaa2b/src/max-ten.js#L75 https://github.com/textlint-ja/textlint-rule-max-ten/blob/3dfb5f458d170079401c5a501553ce46c59eaa2b/src/max-ten.js#L89 https://github.com/textlint-ja/textlint-rule-max-ten/blob/3dfb5f458d170079401c5a501553ce46c59eaa2b/src/max-ten.js#L97 この辺でハードコードされている部分をオプションとして渡せるようになれば、対応できそうな気がしますね。

次のようなイメージのオプションがあるとよさそうなのかな。 , の組み合わせ をサポートするなら個別指定でよさそうな気がしますが、こういう書き方をするパターンはあるのかな。

{
  "rules": {
    "max-ten": {
        "max" : 3,
        "touten": ",",
        "kuten": "。",
    }
  }
}

📝 sentence-splitterが古いのでアップデートする必要がありそう。

nrkt commented 3 years ago

迅速な対応ありがとうございます! 返信が遅くなり申し訳ありません。

上記のようなオプションがあるととてもありがたいです! 自分は「」と「」の組合せだけあればよいと思っていたのですが、他の組合せがないとは言い切れないので上記のようなオプションが良いのではないかと思いました。

azu commented 3 years ago

https://github.com/textlint-ja/textlint-rule-max-ten/releases/tag/v3.0.0 で実装しました。

{
  "rules": {
    "max-ten": {
      // 読点として扱う文字
      "touten": ",",
      // 句点として扱う文字
      "kuten": "."
    }
  }
}

で利用できます。

nrkt commented 3 years ago

ありがとうございます!!とても助かります!!