tsuyoshicho
commented
4 years ago prh本家では /ISO\-/ と事前処理してあるみたいです。
Closed tsuyoshicho closed 4 years ago
tsuyoshicho
commented
4 years ago prh本家では /ISO\-/ と事前処理してあるみたいです。
tsuyoshicho
commented
4 years ago LGTM!
tsuyoshicho
commented
4 years ago @azu すみません、ミスってました。
で /ISO-/ でした...
azu
commented
4 years ago 多分、正規表現のエスケープなのでどちらでも大丈夫ですね。
tsuyoshicho
commented
4 years ago あれ…更新してみても出てたので再確認したのですが…
あとで再度確認しなおしてみます。
tsuyoshicho
commented
4 years ago いえ、やっぱりだめそうです。
確認PRの azu-prh-rules/ja/web+db.yml 有効化前
https://github.com/tsuyoshicho/textlint-template-tech/runs/1205494195?check_suite_focus=true#step:7:16
OK
ですが
確認PRの azu-prh-rules/ja/web+db.yml 有効化後
https://github.com/tsuyoshicho/textlint-template-tech/commit/fe4cccf88a8cdfa53927ff4b313f3e87fb7b2051
https://github.com/tsuyoshicho/textlint-template-tech/runs/1207564281?check_suite_focus=true#step:7:18 は、NGになっています。...
azu
commented
4 years ago 確認ありがとうございます！
2020年10月5日(月) 13:34 Tsuyoshi CHO notifications@github.com:
大丈夫でした、ありがとうございます。
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/azu/prh.yml/issues/5#issuecomment-703390692, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAE2AVO3EVGGQGGCBQYQWDSJFEETANCNFSM4SDWQ7CA .
tsuyoshicho
commented
4 years ago @azu ping すみません、そのメールにある確認ちょっと問題があって、内容を更新しています。 改めて連絡おば。
azu
commented
4 years ago 修正しておきました。
📝 自分はもうこれ使ってなくて確認する方法がCIしかないので、直接修正するPRした方が良いです。
tsuyoshicho
commented
4 years ago そうでしたか...なるほど了解です。 もしなにか他に見付けたときは、PRするかfork版を自分で手直しか、とにかく負担がないようにします。
ありがとうございます。
https://github.com/azu/prh.yml/blob/5cf8448735da01637aee161ff0543e6dcd1e2a2a/ja/web%2Bdb.yml#L790-L791
上記について、正規表現NGとエラーがでました。
via https://github.com/tsuyoshicho/textlint-template-tech/pull/1/checks?check_run_id=1205413411#step:6:18
外した後:成功 https://github.com/tsuyoshicho/textlint-template-tech/pull/1/checks?check_run_id=1205462194