レビューは最優先にするようにしている

社内でやったLT資料です

Transcript

1. レビューは最優先で やるようにしている 2023-12-14 ひむらともひこ
2. 自己紹介 ひむら ともひこ 最近遊びたいこと(希望) • Linux Desktopの設定(NixOS) • 要求の管理
3. レビューはなるべく 最優先でやるようにしている
4. レビューされない場合は だいたいやり忘れているので 言ってください
5. なぜ?
6. 組織パフォーマンスを落とす要因
7. リードタイムを無駄に長くする
8. 私のリードタイムの大体の定義 デリバリー レビュー CI 実装 企画 起案 リードタイム 変更リードタイム 本当に短くしたい時間 でも、メトリクスとしては揺れ幅が大きく使いづらい メトリクスとして代わりに使う時間 完成してからユーザに届くまでの時間 できてるのにユーザに届かないのは無駄
9. 変更のリードタイムと人間の関与 デリバリー レビュー CI 自動化されている 自動化されている 待たせた分だけ長くなる
10. 変更のリードタイムが長くなる要因は レビューしないせいで起きうる CIが長かったり、デリバリーが自動化されてなかったり、 承認が必要だったりする場合は誤差になる場合はある。
11. まとめ
12. 組織のパフォーマンスをあげたい • 変更のリードタイムは短くなって欲しい ◦ できてるのにユーザに届かないのは、できてないのと同じ • レビューが遅くなると遅くなった分だけ変更のリードタイムが伸びる ◦ 1日で開発したものが2日レビューされないものすごく無駄 • 自分の仕事が遅くなるが組織のパフォーマンスはむしろ良くなる ◦ ほんとに?
13. レビューはなるべく 最優先でやるようにしている 2
14. 自分の仕事が遅くなっても 組織のパフォーマンスは上がるから
15. ある種の理想のフロー CD レビュー待ち CI 実装 レビュー Aさん Bさん
16. X1 ある種の現実。自分の仕事優先 CD レビュー待ち CI 実装 Aさんのレビュー Aさん Bさん 実装 CI Bさんのレビュー レビュー待ち CD Aさんの作業の無駄な時間 = Bさんの実装時間 - Aさんの実装時間 Bさんの作業の無駄な時間 = 0 無駄な時間
17. O2 現実的なある種の理想 CD レビュー待ち CI 実装 Aさんのレビュー Aさん Bさん 実装 CI レビュー待ち CD Aさんの作業の無駄な時間 = 0 Bさんの作業の遅延 = Aさんのレビュー 実装 Bさんのレビュー
18. 例示は理解の試金石 変更のリードタイムを伸ばすもの(ロス) - 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分
19. 例示は理解の試金石 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分
20. おわかりいただけただろうか
21. まとめ 変更のリードタイムという観点でみると - レビューが遅くなるのはマジでやばい - なるべく待たさないでやろう