明日からできるコードレビューの心構え

Transcript

1. 〜 チームの生産性を高めるコードレビューのTIPS 〜 明日からできるコードレビューの心構え @akkiee76
2. 自己紹介 Akihiko Sato / 株式会社ラクス Lead Engineer / @akkiee76 SaaS 開発 (Backend, Frontend) / Mobile 開発 (iOS, Android) 上流工程、コードレビュー、チームの課題改善など 読書 / コーヒー / HHKB / 腹筋ローラー
3. 今日伝えたいこと コードレビューの心構えを知ることで より気持ちのいいレビューができる！ （主にレビュアー向け）
4. コードレビュー技術について ・オブジェクト指向で類型化するコードレビュー ・レビューガイドラインで技術力を見える化する
5. 実際の現場では・・・ 実際にドライにコードレビューをしていると メンバー同士の関係もギクシャク ということも少なくないのでは・・・。
6. レビューの現場では、技術的要素が先行してしまい、 攻撃的なコメントをもらった経験がある方も少なくないのでは？？ 現場あるある ・脳死横展開はやめてください ・実装の意味が分かりません。頭大丈夫ですか？ ・私の期待とは違います（のみ）
7. コードレビューで最も大事なこと レビューで最も重要なのは、 「レビュイーに対する思いやり」です 技術の有用性だけではなく、チームメンバーに対して 敬意を表すことがコード品質を高めることに繋がります。
8. Google Engineering Practices Documentation Google が公開しているエンジニアのプラクティス https://github.com/google/eng-practices ・The Code Reviewer's Guide ・The Change Author's Guide
9. How to write code review comments Summary ・思いやりを持つこと ・理由を説明すること ・指摘に対する明確な指示を与つつ、開発者の意思決定のバランスを取ること ・複雑性を説明させる代わりに、コードを単純化するかコードコメントを追加するように開発者 に推奨せよ https://github.com/google/eng-practices/blob/master/review/reviewer/comments.md
10. Chromium Docs - Respectful Code Reviews Do （やるべきこと） ・能力と善意を想定する ・会って話し合う ・理由を説明する ・理由を聞く https://chromium.googlesource.com/chromium/src/+/HEAD/docs/cr_respect.md
11. Chromium Docs - Respectful Code Reviews Do （やるべきこと） ・終わりを見つける ・適度な時間内に返信する ・ポジティブに述べる
12. Chromium Docs - Respectful Code Reviews Don’t （やってはいけないこと） ・人を辱めない ・過激な言い方やネガティブな表現は避ける ・ツールの使用を妨げない ・自転車置き場にしない https://chromium.googlesource.com/chromium/src/+/HEAD/docs/cr_respect.md
13. ・Google Engineering Practices Documentation ・Chromium Docs - Respectful Code Reviews 原文もタメになるので、ぜひご覧ください！
14. まとめ レビュアーの心構えとして大切なこと 　① レビュイーに対して敬意と思いやりを持つ 　② 理由を聞く・説明する 　③ 直接話してみる（リモートもあり！） 明日からのコードレビューでぜひ実践してみよう！
15. ご清聴ありがとうございました