Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
レビューは最優先にするようにしている
Search
Tomohiko Himura
December 14, 2023
0
260
レビューは最優先にするようにしている
社内でやったLT資料です
Tomohiko Himura
December 14, 2023
Tweet
Share
More Decks by Tomohiko Himura
See All by Tomohiko Himura
バイナリ読むのにElixirしてみた
eiel
0
47
アジャイルはさておきMake People Awesomeしたい
eiel
0
150
再考 Fourkeys メトリクス
eiel
2
650
Test mockをSnapshot testする
eiel
0
110
devenvに入門した
eiel
0
85
関数プログラミングの考え方
eiel
1
280
逆コンウェイ作戦はフィードバックループを作るために 逆向きの流れをつくること (5分版)
eiel
0
380
組織のパフォーマンスを高めるために 第1話 学習と文化
eiel
0
220
チームとチームのチーム
eiel
2
2.1k
Featured
See All Featured
ピンチをチャンスに:未来をつくるプロダクトロードマップ #pmconf2020
aki_iinuma
123
52k
[Rails World 2023 - Day 1 Closing Keynote] - The Magic of Rails
eileencodes
34
2.3k
Visualizing Your Data: Incorporating Mongo into Loggly Infrastructure
mongodb
45
9.6k
Measuring & Analyzing Core Web Vitals
bluesmoon
7
460
How To Stay Up To Date on Web Technology
chriscoyier
790
250k
GraphQLとの向き合い方2022年版
quramy
46
14k
StorybookのUI Testing Handbookを読んだ
zakiyama
30
5.8k
Building an army of robots
kneath
306
45k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
252
21k
Writing Fast Ruby
sferik
628
61k
個人開発の失敗を避けるイケてる考え方 / tips for indie hackers
panda_program
106
19k
Designing Experiences People Love
moore
142
24k
Transcript
レビューは最優先で やるようにしている 2023-12-14 ひむらともひこ
自己紹介 ひむら ともひこ 最近遊びたいこと(希望) • Linux Desktopの設定(NixOS) • 要求の管理
レビューはなるべく 最優先でやるようにしている
レビューされない場合は だいたいやり忘れているので 言ってください
なぜ?
組織パフォーマンスを落とす要因
リードタイムを無駄に長くする
私のリードタイムの大体の定義 デリバリー レビュー CI 実装 企画 起案 リードタイム 変更リードタイム 本当に短くしたい時間
でも、メトリクスとしては揺れ幅が大きく使いづらい メトリクスとして代わりに使う時間 完成してからユーザに届くまでの時間 できてるのにユーザに届かないのは無駄
変更のリードタイムと人間の関与 デリバリー レビュー CI 自動化されている 自動化されている 待たせた分だけ長くなる
変更のリードタイムが長くなる要因は レビューしないせいで起きうる CIが長かったり、デリバリーが自動化されてなかったり、 承認が必要だったりする場合は誤差になる場合はある。
まとめ
組織のパフォーマンスをあげたい • 変更のリードタイムは短くなって欲しい ◦ できてるのにユーザに届かないのは、できてないのと同じ • レビューが遅くなると遅くなった分だけ変更のリードタイムが伸びる ◦ 1日で開発したものが2日レビューされないものすごく無駄 •
自分の仕事が遅くなるが組織のパフォーマンスはむしろ良くなる ◦ ほんとに?
レビューはなるべく 最優先でやるようにしている 2
自分の仕事が遅くなっても 組織のパフォーマンスは上がるから
ある種の理想のフロー CD レビュー待ち CI 実装 レビュー Aさん Bさん
X1 ある種の現実。自分の仕事優先 CD レビュー待ち CI 実装 Aさんのレビュー Aさん Bさん 実装
CI Bさんのレビュー レビュー待ち CD Aさんの作業の無駄な時間 = Bさんの実装時間 - Aさんの実装時間 Bさんの作業の無駄な時間 = 0 無駄な時間
O2 現実的なある種の理想 CD レビュー待ち CI 実装 Aさんのレビュー Aさん Bさん 実装
CI レビュー待ち CD Aさんの作業の無駄な時間 = 0 Bさんの作業の遅延 = Aさんのレビュー 実装 Bさんのレビュー
例示は理解の試金石 変更のリードタイムを伸ばすもの(ロス) - Aさんの作業の無駄な時間 - Bさんの作業の遅延 例1 実装時 間 レビュー
に使っ た時間 CIの時 間 CDの時 間 X1 ロス X1リー ドタイム O2 ロス O2 リー ドタイム Aさん 10分 20分 5分 5分 60-10 = 50分 65分 0分 15分 Bさん 60分 5分 5分 5分 0分 30分 5分 35分 合計 95分 50分
例示は理解の試金石 2 例1 実装時 間 レビュー に使っ た時間 CIの時 間
CDの時 間 X1 ロス X1リー ドタイム O2 ロス O2 リー ドタイム Aさん 1日 60分 5分 5分 2日 2日 0分 40分 Bさん 3日 30分 5分 5分 0分 70分 30分 100分 合計 2日 140分
おわかりいただけただろうか
まとめ 変更のリードタイムという観点でみると - レビューが遅くなるのはマジでやばい - なるべく待たさないでやろう