コードレビューのありがち失敗

市ヶ谷Geek★Night #4 しくじりサミット LT

スライドは

http://mshibuya.github.io/slides/review

にてご覧頂けます

自己紹介

渋谷　充宏

• Demand Side Science株式会社所属
  • プロダクトマネージャ
• https://github.com/mshibuya
• Ruby/Rails好き
  • RailsAdminメンテナ
• 最近Scalaはじめました
• 猫好き

コードレビュー、むずくないですか？

利点

• 問題の早期発見
• 知識の共有・教育
• コードのオーナーシップの共有＝脱属人化

ありがち

パターン

• 同じ人ばっかレビューしてる
• 人間関係が険悪に？
• 議論が割れた時どうする？

同じ人ばっかレビューしてる

Pull Requestごとにレビュイーがレビュアーを指名する

• DSSでは以前よりこのスタイル
• 同じチームのメンバーにレビューを依頼するパターンが多い

人間関係が険悪に？

要求しない。質問や提案する

• 絵文字や画像を使って楽しい雰囲気を作っていくのも大事

悪い点だけではなく、良い点を見つけ積極的に讃えてゆく

• 👍したり

議論が割れた時どうする？

レビューコメントへの対応を必須でなく任意にする手がある

• http://blog.smartbear.com/code-review/avoiding-the-politics-of-code-review/

まとめ

楽しくやるのが一番！

ご静聴

ありがとう

ございました！