Skip to content

add separatorChars to options for sentenceSplitter's argument #14

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

Merged
merged 2 commits into from
Jan 3, 2017

Conversation

yujiorama
Copy link
Contributor

no-doubled-joshi のオプションで、文の区切り文字を制御できるようにしたいです。

@@ -68,6 +70,8 @@ textlint --rule no-doubled-joshi README.md
- 下記参照。例外としているものもエラーとするかどうか
- `allow`(default: []) :複数回の出現を許す助詞
- 並立の助詞など、複数回出現しても無視する助詞を指定します。
- `separatorChars`(default: ["。",,"?","!","?","!"]) : 文の区切り文字として使用する文字の配列
Copy link
Member

Choose a reason for hiding this comment

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

,, が2重になってます

Copy link
Contributor Author

Choose a reason for hiding this comment

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

失礼しました…

@azu
Copy link
Member

azu commented Jan 3, 2017

他は問題なさそうです。

@azu azu changed the title add separatorChars to options for sentenceSplitter\'s argument add separatorChars to options for sentenceSplitter's argument Jan 3, 2017
@azu azu merged commit 6afcfa3 into textlint-ja:master Jan 3, 2017
@azu
Copy link
Member

azu commented Jan 3, 2017

https://github.com/textlint-ja/textlint-rule-no-doubled-joshi/releases/tag/3.4.0 リリースしました。

この問題はsentence-splitterに依存する他のルールでも起きる可能性があるので根が深そうですね。

RedPenのように、ユーザー環境を定義する方法が必要になるかもしれないですね。
(ルール自体は修正する必要があるのは同じですが)
http://redpen.cc/docs/latest/index_ja.html#override-language

@yujiorama
Copy link
Contributor Author

ja-no-mixed-period のほうでは全角ピリオドを追加することができていたので、同じように設定できるとわかりやすいですね(ということに今気づきました)。

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