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

やっぱり余白が大切だった話

 やっぱり余白が大切だった話

D-Plus Tokyo #12~個人の工夫をチームの力に!開発生産性向上の文化づくりどうしてる?
https://d-plus.connpass.com/event/346580/
での登壇資料です

KAKEHASHI

March 13, 2025
Tweet

More Decks by KAKEHASHI

Other Decks in Technology

Transcript

  1. ©KAKEHASHI inc. 7 AI在庫管理の開発スタイル • スクラムで1週間スプリントで素早くフィードバックを得る • フィーチャーチームごとに担当機能を明確に分け、チーム内でのコンテキスト の共有を徹底 •

    ストーリーを小さく明確化し、目の前のストーリーの達成に集中できるように する • レビュー待ち時間を作らないよう、モブやペアでの開発を基本とする • フロー効率を優先している
  2. ©KAKEHASHI inc. 10 リソース効率とフロー効率(3/3) リソース効率重視 • 短期的にはスループットが高い • コンテキストスイッチが発生 •

    属人化してしまいボトルネックが発生 フロー効率重視 • リードタイムが短い • 同時にナレッジシェアが行える • 一定の冗長性を保てる https://kakehashi-dev.hatenablog.com/entry/2023/12/02/091000
  3. ©KAKEHASHI inc. 25 どんなPull Requestだったらレビューなしでいけそうか 小さいPull Request だったらレビュー無しで もいいんじゃない? Terraformのコードは

    plan結果が no-changeだったらど んどんやってしまって良 い気がする issue templateとかは やらなくてもいいので は? テスト通ってる=挙動が 変わっていないのは必須 だよね そもそも振る舞いを変え ずに整頓する(本来のリ ファクタリング)のにも練 習が必要かも DBマイグレーションみた いな不可逆な変更はダメ だよね それを確かめるために もどんどん練習してい きたい Tidy First?の1部に書 いてあるような変更内容 だったら良いんじゃな い?
  4. ©KAKEHASHI inc. 26 実際にこんなルールで取り組みを開始しました 1. PR タイトルのプレフィックスを [Tidy] とつけて分かりやすくする 2.

    変更内容が単一である(複数の変更をひとつの PR に含めていない) 3. 修正箇所のテストが存在し、テストが通っている (コメントのみの修正等は除く) 4. アプリケーションの挙動は変わっていないこと 5. 可逆的である (仮に不具合が発生した場合は、この PR を Revert するだけでよい) 6. 不具合が発生するリスクが低い自信を持てている (PR 作成者の自己申告でよい)