Skip to content
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

$1参照で余計な文字削除が行われているように見える事例 #40

Open
tsuyoshicho opened this issue Oct 31, 2020 · 1 comment

Comments

@tsuyoshicho
Copy link

tsuyoshicho commented Oct 31, 2020

link: tsuyoshicho/zenn-articles-and-books#24 (comment)

つかっているルールは

https://github.com/prh/rules/blob/711e00793d9d69eeda04d68a796b6d9afb6d5748/media/WEB%2BDB_PRESS.yml#L1060-L1061

になります。

結果にもとづいて結果に基づて と修正されますが、ルールは

  - pattern: /基([いかきくけこ])|基ず([いかきくけこ])|もとず([いかきくけこ])|もとづ([いかきくけこ])/
    expected: 基づ$1

であり、見える限りで $1 で補うとき、余計に文字を消してしまっているように思えます。

@tsuyoshicho tsuyoshicho changed the title $1参照が正常に行われていないように見える事例 $1参照で余計な文字削除が行われていないように見える事例 Oct 31, 2020
@tsuyoshicho tsuyoshicho changed the title $1参照で余計な文字削除が行われていないように見える事例 $1参照で余計な文字削除が行われているように見える事例 Oct 31, 2020
@tsuyoshicho
Copy link
Author

tsuyoshicho commented Oct 31, 2020

tsuyoshicho/zenn-articles-and-books#24 (comment)

その後でこう指摘されました

https://github.com/prh/rules/blob/711e00793d9d69eeda04d68a796b6d9afb6d5748/media/WEB%2BDB_PRESS.yml#L1066-L1067

  - pattern: /基([いかきくけこ])|基ず([いかきくけこ])|もとず([いかきくけこ])|もとづ([いかきくけこ])/
    expected: 基づ$1

  - pattern: /基([^数準盤板底礎本])/
    expected: もと$1

ルールが循環していますね、これは別件として辞書リポジトリにIssueを上げておきます。

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

No branches or pull requests

1 participant