コードレビュー

コードレビューについてのメモ

コードレビューについて

悪い兆候のあるコードに対する抑止力になり、「動けばよい」という妥協を封じ込める
教育的な効果
コードを共有することで、一人が死んでも大丈夫的な効果
コミュニケーション的な効果

コードレビューに対するコスト

コピー＆ペーストのコード(=>変更に弱くなる)
1つの関数やメソッドが長過ぎるコード(=>見通しが悪く、バグを生みやすい)
テストコードが存在しなかったり、通らないコード(=>保守性が低まる)

コミットを綺麗にする

git rebase -i HEAD~3 // とかでコミットを綺麗にしてpushするとか

バグはレビュアでもコミッタの責任でもなくチームの責任なので、思ったことをコメントする
違う人がレビュアでも気になったら、横やりでコメントする
ピアレビュー = 同僚などフラットな間柄で互いにレビューしあう

上記みたいな詳細な説明が見つけられなかったので、以下は

みてください。