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

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

8fea11f3454c94cb5a5599ff03021471?s=47 SHIOYA, Hiromu
November 15, 2021
Technology
3
3.4k

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

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

8fea11f3454c94cb5a5599ff03021471?s=128

SHIOYA, Hiromu

November 15, 2021
Tweet

More Decks by SHIOYA, Hiromu

See All by SHIOYA, Hiromu
リモートワーク時代のコミュニケーションデザイン : コーチングに学ぶ1on1の基礎/Learn Basics of 1on1 from Coaching
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
1
970
The Show Must Go On 終わりのない道のなかばでふりかえる俺とDevLOVE / The Show Must Go On Looking back of me and DevLOVE in the middle of the endless road
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
0
380
プロフェッショナルなエンジニアのためのコミュ力概論/Overview of communication skills for professional engineers
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
2
2.2k
登壇は最強の生存戦略である / Speaking in conferences is the best survival strategy
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
0
3.1k
礼節から育てるチームの健康と信頼性 / Improve team health and reliability starting with civility
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
8
15k
塹壕よりDevOpsとSRE / DevOps and SRE from the Trenches
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
4
4.5k
チームで働くための礼節と心理的安全性 / Civility and psychological safety for working in a team
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
22
23k
3つの「Re」〜ソフトウェアの信頼性を高めるためにぼくたちができること〜 / What we can do to improve the reliability of the software
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
0
940
チームで働くための知識体系 / Body of knowledge for working in a team
8fea11f3454c94cb5a5599ff03021471?s=48 kwappa
8
8k

Other Decks in Technology

See All in Technology
HTTP Session Architecture Pattern
484571ccba0db66b69dc11761afdd0c4?s=48 chiroito
1
390
組織でPower Virtual Agentsを導入するために知っておきたいこと
C0c0e8e4d7f83912cb1b1a0699df82a5?s=48 miyakemito
0
1.6k
1,000万人以上が利用する「家族アルバム みてね」のSRE組織は4年間でどのように作られてきたのか/SRE NEXT 2022
46839cf590a549efe13547c17a6b2fde?s=48 isaoshimizu
4
2.7k
JAWS-UG 朝会 #33 登壇資料
08dd0b93e011904f40d60d5a1b54c671?s=48 takakuni
0
370
Devに力を授けたいSREのあゆみ / SRE that wants to empower developers
E12b5afe5b4e6552019c09d6ac4128c3?s=48 tocyuki
3
460
220428event_overview
E55fbffb4afd0a84d36c945ed68d8c19?s=48 caddi_eng
2
210
様々な現場のPower Platform ～小さなエンジニアの奮闘記～
39adbc0cddb8c0ea8470c80248e9e71b?s=48 hyodol2513
0
1.9k
街じゅうを"駅前化"する電動マイクロモビリティのシェアサービス「LUUP」のIoTとSRE
1333e8256b0ba80c369f4bcc98a839d7?s=48 0gm
1
510
Stripe Search APIを利用した、LINEとStripeの顧客情報連携/line-dc-202205
E77ca94266ffd3d69f8aee91f7468264?s=48 stripehideokamoto
0
120
僕の Microsoft Teams (+α) 便利技紹介 2022年春
54bcc1069d75636cc735e72877dd30ec?s=48 taichinakamura
0
2.7k
⚡Lightdashを試してみた
2c6a9bc003b77685330cc9359b6fbbc4?s=48 k_data_analyst
0
160
ZOZOTOWNのProduction Readiness Checklistと信頼性向上の取り組み / Improvement the reliability of ZOZOTOWN with Production Readiness Checklist
08672189228a165c14ed8d896f9a4d3a?s=48 akitok_
5
1.5k

Featured

See All Featured
Visualization
170b760e0147792d0140e59ec78e9893?s=48 eitanlees
124
11k
The Art of Programming - Codeland 2020
719435d98d452de7ac367c828266cf01?s=48 erikaheidi
31
5.8k
GraphQLの誤解/rethinking-graphql
3cb4e761dbdb7aebd1ffd85f752e1e3e?s=48 sonatard
24
6.2k
The MySQL Ecosystem @ GitHub 2015
Be9caeb9d4ef9944d151af909063ed6e?s=48 samlambert
238
11k
XXLCSS - How to scale CSS and keep your sanity
1b8ad785acdd1ce1c99914b1c2a4e10e?s=48 sugarenia
236
1M
The World Runs on Bad Software
20bfe76b3d6105641f879fe45cfc9272?s=48 bkeepers
PRO
56
5.2k
For a Future-Friendly Web
F68cb25ae3e5c2106997454b13b7737d?s=48 brad_frost
164
7.4k
StorybookのUI Testing Handbookを読んだ
50fc0fdc2327ecbe7350645812de52e3?s=48 zakiyama
4
2k
Docker and Python
Ecdea9b9714877b86cee08458f085481?s=48 trallard
27
1.5k
A Philosophy of Restraint
C9b18d9dff88a9dd2393364c2b3b21bd?s=48 colly
192
14k
Design by the Numbers
027d1ebf66cc039a0bd3b55eeadbe75d?s=48 sachag
271
17k
Gamification - CAS2011
4394ceb8b277f35ee3da7030384efb5d?s=48 davidbonilla
75
3.9k

Transcript

  1. レビューのつらさを倒したい 2021/11/05 SHIOYA, Hiromu / @kwappa

  2. レビューとはなんじゃ ここでは「仕事の検証、見直し」 エンジニアは「コードレビュー」をよくやる 他の職種でもあると思うのでよしなに解釈してください

  3. 用語の確認 レビュアー レビュー「する」人 レビュイー レビュー「される」人 つまり「レビューをお願いする」人

  4. 最近よく聞くつらみ レビュー「する」のがつらい PRが来る→レビューしなきゃ レビュー「してもらう」のがつらい レビューお願いしたい＊＊さんが忙しそう

  5. レビューがつらいことによる害 価値を届けるまでのリードタイムが増える とはいえノーチェックで出すのはまずいが…

  6. そのレビュー意味あんの？ トリプルチェックの弊害 #現場猫 実写版 実写版・現場猫その5 トリプルチェック。

  7. レビューの流れ レビュイー なにかコードを書く Pull Requestでレビューを依頼する レビュアー レビューの依頼が届く レビューする Approve する

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

    = Looks Good To Me : 「よさそう」 つい「承認」だと思いがち レビュイー : だってレビュアーが 承認 したじゃん レビュアー : 絶対ヨシ

  9. なぜレビューがつらいのか レビュアーがなんにもうれしくないから レビューの時間は自分のタスクが進まない 承認 する責任がある？ 不具合があったら自分の責任？ うれしくないから後回しにする レビュイーもつらい プロダクトもつらい

  10. こんなのもうイヤだ 仕事のやり方を変えよう

  11. その1 : 責任所在 レビュイー : タスクの責任者 仕事に責任を持つ 不安を解消するためにレビューを依頼する レビュアー :

    タスクのサポーター レビュイーの不安を解消する手助けをする 知識や経験でタスクの品質向上に 貢献 する 承認 じゃないよ

  12. その2 : 時間配分 レビューは必要 なので業務の時間に織り込む なのでリードタイムに織り込む

  13. その3 : プロセス改善 プロセス改善？ レビューの目的を明確に レビューのサイクルを短く レビュイーが責任を持つ

  14. レビューの目的を明確に レビュイー : 不安を取り除く タスクを説明する 目的は何か、そのために何をしたか、自分で何を確認したか 確認しやすい粒度でレビューを依頼する 不安を言語化する 見てほしい、知恵を貸してほしいポイントを明示 不安を解消する

    レビューをもとにタスクの品質をあげる →手直し上等

  15. レビューの目的を明確に レビュアー : 貢献する レビュイーの不安を取り除く そのための approve = 賛成 知識・経験からのコメント

    タスクの品質が向上するために レビュイーが成長するために モヒカンの手斧はいらない

  16. レビューのサイクルを短く レビューが通れば完了 例 : 基本設計→詳細設計→実装→試験→レビュー テストコードまで書いてレビュー依頼 「そのテーブル名いまいちだよね」→ レビューの範囲を小さくする 例 :

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

  17. レビュイーが責任を持つ タスクをdoneにするのはレビュイーの 責任 レビューの単位を小さくする レビューするべき箇所を明示する そのためのサポートをするのがレビュアーの 貢献 レビューを溜め込まない 当然発生する業務として織り込む 不安を取り除く

    知識と経験 態度や言葉遣い

  18. まとめ : レビューをつらくなくしよう レビュイーが責任を持つ レビュアーは貢献する お互い協力する →最適な結果のために