Fix check_coding_rule workflow

[sksat]

概要

check_coding_ruleのCIが落ちるべき時に落ちていなかったので修正

Issue

• インクルードガードのチェックを有効に #234 で発生

詳細

• インクルードガードのチェックを有効に #234 (comment)

[sksat]

やっぱりそういう挙動なんだな．まあコーディングルールのチェックはこれでいいんじゃないでしょうか

[sksat]

他の自前でreviewdog叩いてるやつもfilter_modeを念の為fileぐらいにしておいてもいいかもしれませんね

[meltingrabbit]

この， check_coding_rule はexit 1 を吐いてるけど，表示的には pass してるのは，いいんだっけ？

[meltingrabbit]

actions，最終的になにをすると落ちるのかが分かってない（stepのどこか1つでもexitが0でなければおちる？ 次のstepを実行するかどうかはどこでみてる？）

[meltingrabbit]

最後のreview dogを抜くと，pythonスクリプトで落ちてても，CI通っちゃうね．

CIの結果は，pythonスクリプトの結果にしたほうが良い？

https://github.com/ut-issl/c2a-core/runs/5046076282?check_suite_focus=true

同様に filter-mode=added でも，通っちゃう．（review dogが刺さらないため）

https://github.com/ut-issl/c2a-core/runs/5046098788?check_suite_focus=true

[meltingrabbit]

うー

filter-mode=file でもCIとおるなぁ．

https://github.com/ut-issl/c2a-core/runs/5046110415?check_suite_focus=true#step:8:1

[meltingrabbit]

actions，最終的になにをすると落ちるのかが分かってない（stepのどこか1つでもexitが0でなければおちる？ 次のstepを実行するかどうかはどこでみてる？）

continue-on-error: true でみてるのか，理解．

[meltingrabbit]

@sksat

鈴本の所感まとめると

• filter-mode が nofilter 以外では，結局diffがでず，reviewdogがささらない（たとえ file でも）ので， nofilter で運用するしかない
• 現在，CI scriptの返り値ではなく，reviewdogの返り値を見ているので，reviewdogがささらないなら，CIは落ちない
• すなわち，nofilter で運用する他ない
• nofilter なら，そもそも reviewdogいらなくない？とおもったり

って感じなので，CIスクリプトの方針決まったらそれにして，今通ってないのを僕の方で直すので，その後マージしましょう．

[sksat]

nofilterだったらreviewdog要らないのではというのはどこから来てます？あと，そもそもこれは元のPRがcheck_coding_ruleのCIの方を弄っているから(かつ，developが壊れていることに気付いていなかったから)起きている問題(diffが出ない，というやつ)であって，普段はなんも困らないはずです．
やるとしたらスクリプトのexit statusを保存しておいて最後に絶対その値でexitするstepを入れるとかかなと思っています．reviewdogをスクリプトの後に走らせたい(が，スクリプトがexit 1してもreviewdogは走ってほしい)という要求があるだけなので．

[meltingrabbit]

nofilterだったらreviewdog要らないのではというのはどこから来てます？

nofilterの場合，落ちてる理由を見るのはstatusみるか，CI結果ログ見るかなので，revrewdog入れても入れなくても同じ？と思った感じ（たしかにstatusにはでるが，落ちてたらそっち見るのではなく↓のCI結界一覧の details 見るのではないかなぁ．．．とおもって

[meltingrabbit]

普段はなんも困らない

たしかに，CI側をいじらなければ良いので，よいか．

スクリプトのexit statusを保存しておいて最後に絶対その値でexitするstepを入れるとかかなと思っています．

安全なのはそれよね．

一旦，ここではこれでマージして，別でこれやりますか？

[sksat]

個人的にはfilter-modeどうしようがまずUnchanged files with check annotations Beta見るのでそうかな？というかんじですね > nofilter

[sksat]

ちゃんとexitするstep追加するのはすぐできるんでやっちゃいます

[sksat]

ん？CI走らない．なぜ

[sksat]

少々手こずりましたがこれでよさそう

[meltingrabbit]

ありがとー．じゃあCI通るようにします．

こうなれば，reviewdogの返り値気にしなくて良くなるので，また filter をaddedとかにもどします？

[sksat]

まあaddedでいいとは思います

[sksat]

テストコミットだけ消しますね

[meltingrabbit]

これでいいのでは？

[sksat]

お

[sksat]

ではマージします