コードレビュー
コードレビューについてのメモ
コードレビューについて
メリット
悪い兆候のあるコードに対する抑止力になり、「動けばよい」という妥協を封じ込める
教育的な効果
コードを共有することで、一人が死んでも大丈夫的な効果
コミュニケーション的な効果
###デメリット
コードレビューに対するコスト
悪い兆候
コピー＆ペーストのコード(=>変更に弱くなる)
1つの関数やメソッドが長過ぎるコード(=>見通しが悪く、バグを生みやすい)
テストコードが存在しなかったり、通らないコード(=>保守性が低まる)
レビューの心得
コミットを綺麗にする
git rebase -i HEAD~3 // とかでコミットを綺麗にしてpushするとか
バグはレビュアでもコミッタの責任でもなくチームの責任なので、思ったことをコメントする
違う人がレビュアでも気になったら、横やりでコメントする
ピアレビュー = 同僚などフラットな間柄で互いにレビューしあう
レビューの手法
上記みたいな詳細な説明が見つけられなかったので、以下は
みてください。
-
チームレビュー
-
ウォークスルー
-
ペアレビュー
-
パスアラウンド
-
アドホックレビュー
参考URL
書籍
レビューツール
diffコメント型
Rietveld
Gerrit Code Review
Review Board
Barkeep
統合型
Gitlab
RhodeCode
Phabricator
GitHub
Bitbucket
Chromium
Lhaz