【はてな×ZOZO】プルリクエスト分割が開発生産性向上のカギ？〜エンジニアチーム全体で改善を推進〜 #開発生産性_findy

komitake @komitake

参加します 【はてな×ZOZO】プルリクエスト分割が開発生産性向上のカギ？〜エンジニアチーム全体で改善を推進〜 findy.connpass.com/event/276072/ #開発生産性_findy

ことみん @kotomin_m

見てます！！ 【はてな×ZOZO】プルリクエスト分割が開発生産性向上のカギ？〜エンジニアチーム全体で改善を推進〜 findy.connpass.com/event/276072/ #開発生産性_findy

ham @hamchance0215

参加してます〜 【はてな×ZOZO】プルリクエスト分割が開発生産性向上のカギ？〜エンジニアチーム全体で改善を推進〜 findy.connpass.com/event/276072/ #開発生産性_findy

komitake @komitake

へ〜 ZOZOは出戻り組がそれなりにいるらしい #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

ZOZO御立田さん シンプルな目標を立てた「生産性3倍にしようぜ」 →具体的に達成に向けてアクションする時に「平均プルリククローズ時間：24時間」「プルリク作成数」を見ていた #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

はてな 五十嵐さん 中長期的な目標は「アクティブユーザー数を増やす」こと 本格的に追い始めたのはリファクタリングが終了してからだった #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

はてな五十嵐さん 「デュアルトラックアジャイル開発」を進めていくときにも生産性向上は意識しながらやっていた #開発生産性_findy

西から来た馬づらの男@DevRel/Japan CONFERENCE 2023 @beppu01

始まりました〜 見える化 数値化の取り組み 【はてな×ZOZO】プルリクエスト分割が開発生産性向上のカギ？〜エンジニアチーム全体で改善を推進〜 findy.connpass.com/event/276072/ #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

ZOZO御立田さん 開発生産性の可視化が必要になったタイミングでFindy Team+を眺めるようになった。 世の中の数値と比較したら「もっとやらなきゃ」となった チーム内だけで数値を眺めていても、遅いかどうかは分からなかった #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

ZOZO御立田さん 「課題が何か？」が分からないことが課題だった ・なんでこの時間になってしまっているの？ ・「遅い」とチームで認識して、「何をやっているから遅くなっているのか？」を話し合えるようにする必要があった #開発生産性_findy

komitake @komitake

"課題に気づけていないことが課題" #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

ZOZO 御立田さん 課題を一気に解決したわけではない （1個課題が解決されると、新しい課題が見えてくる感覚だった） 取り込めることを徐々にやっていったので、取り組みハードルは高くなかった #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

ZOZO 御立田さん 「プルリクを小さくしていく」動きは、チーム内では理解があった でも会社全体でみると「1個の大きいプルリクを5個に分割して小さくするだけじゃ意味がないのではないか？」は意見としてもらったことがある →小さいプルリクにする意義を説明した #開発生産性_findy

ことみん @kotomin_m

プルリクを分けると修正の舵を切りやすい、これなわかる〜〜 #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

ZOZO 御立田さん 小さくするメリット：プルリクの舵を切れる ・大きくプルリク出してしまうと、修正事項があったときにまるごと修正しなくてはならない ・小さく出しておけば、修正事項のあるプルリクだけ直せば良い（プルリク作成者も嬉しい） #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

はてな五十嵐さん プルリクを小さくするには、タスクの全体感を把握して細分化する必要がある プルリク小さくすることは品質の向上にもつながる #開発生産性_findy

SRAUFactory@リモートワーク @SRAUFactory

> プルリクの分割するのに全体が見えてないと分割できない これはよくわかりますね！ ただ細かく分割しすぎるとCIのリソース枯渇するので、悩ましいですね。 最近、CIのリソース周りでトラブった経験もあるので。 #開発生産性_FIndy

ずーみん@Findy Team+ @osaemaru08

はてな五十嵐さん 1000行のプルリクと100行のプルリク、どちらをレビューしたいか？ →答えは基本的に同じになるはず #開発生産性_findy

komitake @komitake

初めから小さいPRを意識してコーディングする人と、没頭してガーッとコーディングしていくとかなりの実装量になるので後から分割する人といる気はする。後から分割は意外とスキルいりますよね #開発生産性_findy

ことみん @kotomin_m

「プルリクは小さい方が絶対楽でしょ」これ分かるようになれば当たり前なんだけど、これ理解までが難しいよね 理解するためのサポートが必要だと思う #開発生産性_findy

SRAUFactory@リモートワーク @SRAUFactory

ZOZOの改善のサイクルタイムの分析データ、凄いですね。。。 techblog.zozo.com/entry/operatio… サイクルタイムの平均値75%OFFって相当な改善かなと思いますよ。 #開発生産性_FIndy

komitake @komitake

なるほど、先にJiraでサブタスク化しちゃうんですね #開発生産性_findy

ずーみん@Findy Team+ @osaemaru08

ZOZO 御立田さん コミット〜オープンまでの時間 ・前提、エンジニアの力量も関係するので、一定時間がかかる ・見積もり時は仕様を見て何となくのチケット分割しかできないので、開発中にJiraのサブタスクを利用して分割し直したりしてた #開発生産性_findy

SRAUFactory@リモートワーク @SRAUFactory

データ見ていると「オープンからレビューまでの平均時間」が0.3hっていうのがすごいですね。 やはりPRすぐレビュー出来るのは大きいんですかね。。。 #開発生産性_FIndy