textlint-rule / textlint-rule-prh

textlint rule for prh.
MIT License
82 stars 6 forks source link

fix: correct `?=` RegExp #12

Closed azu closed 7 years ago

azu commented 7 years ago

Add test case for ?= RegExp. Current behavior has bug.

Original Bug reports: https://gist.github.com/cu39/8999ef8d0794d47da30917aa9e287a6f#file-prh-yml-L9

先読み正規表現のときの挙動がバグっている。 具体的にはここでreturnされている。 https://github.com/azu/textlint-rule-prh/blob/67a43f989850521b820151a0a0b0afd7c161c924/src/prh-rule.js#L83

prh側の置換実装を使ったりできるのかな? applyChangeSetを持ってきた実装を追加

TODO