search

textlint-ja / textlint-rule-no-mix-dearu-desumasu

textlint rule that check no mix である and ですます.
MIT License
27 stars 7 forks source link

Issue 21 エラーメッセージの改善 #25

Closed KEINOS closed 4 years ago

KEINOS commented 4 years ago

Issue https://github.com/textlint-ja/textlint-rule-no-mix-dearu-desumasu/issues/21 対応開始の意思表示のため Draft PR をあげました。

当方の開発環境に Node.js をローカルで入れられないため Dockerfile を暫定的に追加しましたが、Draft を解除（レビュー＆マージ依頼）する際に削除して PR いたします。

Node.js 初心者＆初 node.js の PR です。途中、気になる点や問題ありましたらご指摘いただければ嬉しいです。

KEINOS commented 4 years ago

@azu さま

エラー発生（要修正箇所）の行番号・列番号の表示はレベルが高すぎて実装できませんでした。 取り急ぎ、メッセージ内容の改善提案だけ PR いたします。

おてすきにレビューをお願いいたします。

azu commented 4 years ago

PRありがとうございます！ https://github.com/textlint-ja/textlint-rule-no-mix-dearu-desumasu/releases/tag/4.0.1 としてリリースしました

KEINOS commented 4 years ago

@azu

マージありがとうございます！やった！

KEINOS commented 4 years ago

おっとー、README.md がメッセージ内容の変更に追随してませんでした。別途 PR いたします。