コードレビュー
This content is a draft and will not be included in production builds.
コードレビュー
Section titled “コードレビュー”https://twitter.com/Linda_pp/status/1171416072894894086
Google のコードレビューガイド.レビューコメントに大事なのは
- 優しく
- 理由を説明する
- なるべくどう変更したら良いかまで書く
- 複雑なコードは口頭で説明させるだけでなくコード内にコメントを残させる(かコードを簡潔に直させる)
Google Engineering Practices Documentationが原本。日本語訳もある。
https://twitter.com/daiksy/status/1197023963446050816
ぼくはなんとなくわかるんだけど、コードレビューで自分がつらくなるのは、「問題対わたしたち」も「コードと人格は別」も充分わかっていて、かつその上でコードレビューの指摘内容により自分自身の無能感が浮き彫りになるということがあると思う。
レビューする側の億劫さについてはレビューする際の心理的障壁に書いた。
どのくらいの大きさなら許せるのかについてはPull Requestの適切な大きさも書いた。
上のようなことがレビューが爆速になる!レビュアーに優しいPull Requestの極意に全部書いてある。