Upgrade to Pro — share decks privately, control downloads, hide ads and more …

レビューのつらさを倒したい/To Defeat the Pain of Reviewing

レビューのつらさを倒したい/To Defeat the Pain of Reviewing

2021/11/05 SHIOYA, Hiromu / @kwappa
社内向け講演「レビューのつらさを倒したい」

Hiromu Shioya

November 15, 2021
Tweet

More Decks by Hiromu Shioya

Other Decks in Technology

Transcript

  1. Approve : 承認? approve 1.〈…を〉よいと認める,〈…に〉賛成する. 2.〈…を〉(正式に)承認する,認可する. レビューにおける approve は「賛成」 LGTM

    = Looks Good To Me : 「よさそう」 つい「承認」だと思いがち レビュイー : だってレビュアーが 承認 したじゃん レビュアー : 絶対ヨシ
  2. その1 : 責任所在 レビュイー : タスクの責任者 仕事に責任を持つ 不安を解消するためにレビューを依頼する レビュアー :

    タスクのサポーター レビュイーの不安を解消する手助けをする 知識や経験でタスクの品質向上に 貢献 する 承認 じゃないよ
  3. レビューの目的を明確に レビュアー : 貢献する レビュイーの不安を取り除く そのための approve = 賛成 知識・経験からのコメント

    タスクの品質が向上するために レビュイーが成長するために モヒカンの手斧はいらない
  4. レビューのサイクルを短く レビューが通れば完了 例 : 基本設計→詳細設計→実装→試験→レビュー テストコードまで書いてレビュー依頼 「そのテーブル名いまいちだよね」→ レビューの範囲を小さくする 例 :

    基本設計→レビュー→詳細設計→レビュー→… 不安を小さい単位で解消する よりよいやり方を議論する →これがペアプロ / モブプロの原点