Skip to content
This repository has been archived by the owner on Feb 26, 2022. It is now read-only.

Latest commit

 

History

History

review

Folders and files

NameName
Last commit message
Last commit date

parent directory

..
 
 
 
 
 
 
 
 
 
 
 
 

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

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

スライドは

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

にてご覧頂けます

自己紹介

渋谷 充宏

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

利点

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

ありがち

パターン

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

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

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

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

人間関係が険悪に?

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

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

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

  • 👍したり

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

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

まとめ

楽しくやるのが一番!

ご静聴

ありがとう

ございました!