Tsukuru Tanimichi

うーん、どうなんですかね。 https://github.com/chooyan-eng/code-your-ruby/pull/5#discussion_r171495431 なんかは検知できても良い気がしたんですけど、どうなんだろ、SideCI に詳しくないので分からないです。。

cc: @chooyan-eng 試しにレビューしてみましたけど、いまいちピンと来なかったです。 これがもし職場の OJT だったらデータモデリングから見直して 解答例というか見本となるようなコードを示したりするんですけど そこまでするのも正直しんどいよなぁという感じです。 例えば「このメソッド名はこういう理由で良くないですよ」ってコメントしても、 その背景となる DSL の考え方だったりとか、 責務についての考え方なりの諸々の解説をしないと結局伝わらないと思うんですけど GitHub のコードレビューでそこまでするのもしんどいですしねえ

> ピンと来ない理由 1 に近いですかね。どこまで深くレビューすべきなのかなっていうのがふわっとしてて。あまり表面的なレビューだけしても価値が薄いでしょうし > 結局コードレビューするのがツラいから今までの状況につながるのかな FizzBuzz の PR をしてる人が多いですが、FizzBuzz のコードなんかを見ても「ふーん」とか「へえ」くらいの感想しか出てこなくてレビューできる気がしなかったので、この PR のレビューをすることにしました。

Do you have any update on this? Has anyone reviewed this patch?