エラーメッセージの改善

[azu]

箇条書き: "である"調 と "ですます"調 が混在
=> "ます" がですます調
Total:
である  : 1
ですます: 2

イマイチ意図がわかりにくい。
何を直したらこのエラーが消えるかを明示する

[KEINOS]

どちらが対象であるかを明示した方がいいかと思います。

箇条書き: "である"調 と "ですます"調 が混在
=> 箇条書きは "ですます"調 に統一してください。
Total:
である  : 1
ですます: 2

[azu]

textlint-rule-no-mix-dearu-desumasu/src/MixedChecker.js

Lines 123 to 141 in b0ff44a

	outputMessage(token) {
	const overType = this.getOverType();
	if (overType === "である") {
	// である優先 => 最後の"ですます"を表示
	return `"である"調 と "ですます"調 が混在
	=> "${token.value}" がですます調
	Total:
	である : ${this.dearuCount}
	ですます: ${this.desumasuCount}
	`;
	} else if (overType === "ですます") {
	// ですます優先 => 最後の"である"を表示
	return `"である"調 と "ですます"調 が混在
	=> "${token.value}" がである調
	Total:
	である : ${this.dearuCount}
	ですます: ${this.desumasuCount}
	`;
	}

preferを指定している場合と指定していない場合でメッセージは別々にした方がいいとは思います。

• preferが指定してある場合は、「統一してください」でよさそうです
• preferが指定してない場合は、利用してる数で優先する書き方を出してるだけなので、事実だけを表示したほうが良い気がします。

---

このプロジェクトのCode of Conductはこちらにあるので参照してください。

https://github.com/textlint-ja/textlint-ja/blob/master/CODE_OF_CONDUCT.md

[KEINOS]

確かに prefer 指定がないのに「統一」はおかしいですね。また、「箇条書きは〜に統一してください」の「箇条書き」も、引っ張ってくるのも修正箇所が多そうなので、以下のような文面ではどうでしょう？これであれば、私も PR のお手伝いできるかも。

箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるかは、" 
Total:
である  : 1
ですます: 2

本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "です" 
Total:
である  : 3
ですます: 1

本当は、できれば -f pretty-error 時にも表示される該当行と文字位置も出せればより親切だと思うのですが、オブジェクトのどのプロパティを引っ張ってくればいいのかわかりません。

箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるかは、" (3行目19文字目)
Total:
である  : 1
ですます: 2

本文: "である"調 と "ですます"調 が混在
=> "である"調 の文体に、次の "ですます"調 の箇所があります: "です"  (4行目20文字目)
Total:
である  : 3
ですます: 1

[azu]

行数の表示はformatter側の仕事なのでルール側では出さなくていい気がしますね。
(データとして行数などは入ってるので、メッセージとして入れてしまうと2重になるため)