レビュワーがレビューしやすくなるよう、レビューの方針を明確にする

[chooyan-eng]

レビュワーがレビューしやすくなるよう、レビューの方針を明確にする

[ttanimichi]

表面的なレビューの部分は @sideci とかで刈り取れるかもしれないですねえ

[chooyan-eng]

考えられる解決策として、

1. リポジトリとしてレビューの指針を決める
2. プルリクを出すレビュイーごとに、どんな観点で見てほしいかを明記するようにする
3. 各レビュワーが都合の良いレベルで自由にコメントする

あたりが考えられるかと思います。なお、現状はどちらかと言えば３の想定になっています。

[chooyan-eng]

@ttanimichi
SideCIは一度試してみたのですが、すべてのプルリクで「問題ナシ」と出てしまって挫折したことがありました、、、ただ、その時は「ちょっと試してみた」程度でしたので、もう少し研究してみます！

ちなみに今回レビューしていただいた @h5y1m141 さんのコードは、SideCIでひっかけられそうな指摘内容はありましたか？参考までに伺いたいです。

[ttanimichi]

うーん、どうなんですかね。 #5 (comment) なんかは検知できても良い気がしたんですけど、どうなんだろ、SideCI に詳しくないので分からないです。。

[chooyan-eng]

@ttanimichi
なるほど！確かに、無駄に super 呼び出すだけのメソッドなら静的解析で分かりそうですね。
SideCIや、他に使えそうなツールがあるか、調査してみます。