概要
コードレビューに否定的な現場の話をYYPHPで出したところ、改善策をいただいたので残します。
コードレビューをしない理由
- レビューは時間がかかるからコストの無駄
時間がかかるからコストの無駄と思われてしまう原因
- レビューに回す時のコード量が多すぎる
対策
- PRの粒度を小さくする
- コミットの粒度を小さくする
- 誰が何をやってるかタスクを明確化する
- PRテンプレートを決めてどういうPRなのかをなわかる状態にしておく
そもそもコードレビューは本当に必要なのか
- やばいコード書く人は普通にいるからコードレビューなりで止めるしか無い
やばいコードとは
- なんでもセッションにデータを突っ込むとか
- 1クラス1万行のコードとか