どうして手を動かすよりもチーム内のコードレビューを優先するべきなのか

2024/12/22 開催の「PHP Conference Japan 2024」（https://phpcon.php.gr.jp/2024/ ）の LT 資料です。

詳細：https://fortee.jp/phpcon-2024/proposal/4d93397b-2256-4a13-8eb2-7a69b3e3b920

Transcript

1. どうして手を動かすよりも チーム内のコードレビューを 優先するべきなのか 2024-12-22 PHP Conference Japan 2024
2. 開発生産性改善のために 「コードレビュー最優先」 よく耳にするはなし ※コードレビューが必須な開発フローを前提とします
3. 「コードレビュー最優先です！」
4. • 今の自分の作業を中断してまで？ 「コードレビュー最優先です！」
5. • 今の自分の作業を中断してまで？ • ちょっ、これ終わるまで待って 「コードレビュー最優先です！」
6. • 今の自分の作業を中断してまで？ • ちょっ、これ終わるまで待って • コンテキストスイッチが...... 「コードレビュー最優先です！」
7. • 今の自分の作業を中断してまで？ • ちょっ、これ終わるまで待って • コンテキストスイッチが...... 「コードレビュー最優先です！」 と いう疑問に答えたい
8. 我々（開発者）の 仕事とは何か？
9. 我々（開発者）の仕事とは？
10. • プルリクエスト（以下、PR）を出すこと？ 我々（開発者）の仕事とは？
11. • プルリクエスト（以下、PR）を出すこと？ • 開発した機能のテストをすること？ 我々（開発者）の仕事とは？
12. • プルリクエスト（以下、PR）を出すこと？ • 開発した機能のテストをすること？ • チケットを DONE のレーンに持っていくこと？ 我々（開発者）の仕事とは？
13. • プルリクエスト（以下、PR）を出すこと？ • 開発した機能のテストをすること？ • チケットを DONE のレーンに持っていくこと？ 我々（開発者）の仕事とは？ △
14. • プルリクエスト（以下、PR）を出すこと？ • 開発した機能のテストをすること？ • チケットを DONE のレーンに持っていくこと？ これらは仕事の「過程」にすぎない 我々（開発者）の仕事とは？ △
15. 突き詰めれば「（誰かにとっての）価値を出すこと」 開発者であれば「機能をリリースすること（※）」 我々（開発者）の仕事とは？ ※1...「"リリース作業"をすること」ではない ※2... リリースしたものに価値があか、という別の論点はあるが今回は割愛
16. で、
17. https://pyama.fun/archives/4751 いい話
18. 仕事は終わらせてはじめて価値になる
19. • 開発者は機能をリリースして はじめて価値（になる可能性が）がうまれる 仕事は終わらせてはじめて価値になる
20. • 開発者は機能をリリースして はじめて価値（になる可能性が）がうまれる • （結果にだけ着目すると） リリースするまでは仕事をしていないのと同じ状態 仕事は終わらせてはじめて価値になる
21. • マージされていない PR • 誰かの動作確認待ちの機能 • 投げたまま返信をもらえていない確認事項 • etc. ...... これまでどれだけ時間をかけたかに依らず まだ価値になっていない仕事（のようなもの）
22. で、本題
23. どうしてコードレビューを優先するべきなのか
24. 「自分の実装の手が止まること」は個の観点 • 自分のタスクを DONE にすることが仕事ではない 他人が待っているものを優先的に終わらせた方がよい • 自分より他人の行動の方がコントロールしにくいから どうしてコードレビューを優先するべきなのか
25. 「自分の実装の手が止まること」は個の観点 • 自分のタスクを DONE にすることが仕事ではない 他人が待っているものを優先的に終わらせた方がよい • 自分より他人の行動の方がコントロールしにくいから どうしてコードレビューを優先するべきなのか
26. • プルリクエスト（以下、PR）を出すこと？ • 開発した機能のテストをすること？ • チケットを DONE のレーンに持っていくこと？ これらは仕事の「過程」にすぎない 再掲・我々（開発者）の仕事とは？
27. 「自分の実装の手が止まること」は個の観点 • 自分のタスクを DONE にすることが仕事ではない 他人が待っているものを優先的に終わらせた方がよい • 自分より他人の行動の方がコントロールしにくいから どうしてコードレビューを優先するべきなのか
28. 自分 同僚 レビュー依頼 作業中 ミーティング 作業中 このようなシチュエーション
29. 自分 同僚 ミーティング 自分の作業を優先した場合 コードレビュー 作業中 次の作業
30. 自分 同僚 ミーティング 作業再開 コードレビューを優先した場合 コードレビュー 次の作業
31. あなたの作業そのものが価値と期待されているとき • 組織内で充分に分業されている • コード等の成果物にお金が支払われる あなたの作業が価値を生むためのボトルネックのとき • 完了するまで他の人の手が止まってしまう作業 逆に、コードレビューを優先すべきでないとき
32. リリース優先で品質を犠牲にして良いという話ではない • 品質を犠牲にすると近い未来にスピードも犠牲になる • そもそも品質を犠牲にしてスピードを上げられるか？ 余談 - リリースすることが仕事だとして
33. リリース優先で品質を犠牲にして良いという話ではない • 品質を犠牲にすると近い未来にスピードも犠牲になる • そもそも品質を犠牲にしてスピードを上げられるか？ how は仕事の目的ではないが 職業倫理上 how に責任を持たなければならない 余談 - リリースすることが仕事だとして
34. 登壇： 寄稿： 岡田 正平／おかしょい X: @okashoi GitHub: @okashoi