## 目的 - コードの品質を高める - 知識伝達・教育 ## 気をつけること(レビュアー編) - 目的を正しく把握する - そもそもの方針が間違っているケースもありある - マインド - HRT(Humility, Respect, Trust / 謙虚, 尊敬, 信頼) - コミュニケーション - 建設的な議論をする。批判する場合明確な理由と代替案を提示する。 - モデル・DB関連 - インデックスの効かないクエリを発行していないか - N+1になっていないか ## 気をつけること(レビュイー編) - 全力を尽くす - その場凌ぎのコードを書かない - セルフレビューをする - スタイルチェック - スペルミス - インデントミス - `""` と `''` が統一されているか - テストが通ることの確認 - PRを丁寧に書く - 変更の目的 - どこを見て欲しいか・不安に思っているところ ## 関連Notes - [[コードレビューを通してドメイン知識を身につける]]