prh/rules

A collection of prh rules

rules's Issues

ルールの循環がある

rules/media/WEB+DB_PRESS.yml

Lines 1060 to 1067 in 711e007

	- pattern: /基([いかきくけこ])|基ず([いかきくけこ])|もとず([いかきくけこ])|もとづ([いかきくけこ])/
	expected: 基づ$1
	- pattern:
	- 元々
	- 本々
	expected: もともと
	- pattern: /基([^数準盤板底礎本])/
	expected: もと$1

もとづく と 基づく をゆききしてしまう。
ルールとしてもまずいのではないか。

see also prh/prh#40

WEB+DB_PRESS.yml: rule invalid for regex

rules/media/WEB+DB_PRESS.yml

Lines 2530 to 2531 in 711e007

	- expected: ISO
	pattern: /ISO-/

上記について、正規表現NGとエラーがでました。

Error while loading rule 'prh': Invalid regular expression: /ISO\-/: Invalid escape

via https://github.com/tsuyoshicho/textlint-template-tech/pull/1/checks?check_run_id=1205413411#step:6:18

make guideline

サマリをここに随時更新していく
初期案はリポジトリに突っ込んでおきました(動かしてないのでコンパイルは通らないかも
https://github.com/prh/rules/blob/master/README.ja.md

techbooster.ymlの「事」を「こと」に修正するルールの改善案

techbooster.yml内に、漢字の「事」をひらくために次のようなルールが定められています。

- expected: こと
  pattern:  /(記|大|仕|返|無|食|見|議|当)?事(?!情|件|前|後|象|例|実|体|態|項|務|業|柄|故)/

正規表現で熟語にはマッチしないようにしようとされていますが、これでは定義されていない熟語（「悪事」や「事変」など）がマッチしてしまいます。
それらの熟語がマッチしないように漢字を追加してもいいのですが、これはホワイトリスト形式そのものの問題であると考えます。
そこで、いっそのことパターンを「前または後を漢字に挟まれていない『事』」という、より広いものに変えるのはどうでしょうか（正規表現でうまく表現できるのかはわかりませんが）。