2021/11/07 DevFest GDG Shikoku

1. Sansan Androidチームが取り組む
コードレビューを早くまわす工夫
Sansan株式会社
技術本部 Mobile Application Group
古川 真次
DevFest Shikoku 2021

2. 自己紹介
● 古川 真次（ふるしん）
● @furusin_oriver
● Sansan,Inc. Android Engineer, Team Leader
● 大阪府在住
● 趣味：勉強会の主催
○ GDG Osaka オーガナイザー
○ Kansai.kt オーガナイザー

3. Sansan Androidチームが取り組む
コードレビューを早くまわす工夫
Sansan株式会社
技術本部 Mobile Application Group
古川 真次
DevFest Shikoku 2021

4. 工夫する前
- レビューが終わらないと後続の実装をプルリク出しにくい
- レビューが終わってないから後続を実装しにくい
- いつ頃レビュー終わりますかね…
- 24時間以内にはレビュー返そうねってルールにはなってた

5. - プルリクを出す側の工夫
- プルリクをレビューする側の工夫
工夫した内容

6. - プルリクを出す側の工夫
- プルリクをレビューする側の工夫
工夫した内容

7. - プルリクのサイズはできれば小さくしよう
- 最大でも２００行までにしよう
- Descriptionを充実させよう
- 動作確認してるはずなので「動作確認した内容」をしっかり書こう
- 「正しく動くこと」は絶対に使うな
- 「正しい」とは？ってなる
- 「◯◯を押したら□□となること」と書こう
プルリクを出す側の工夫

8. Descriptionを充実させよう

9. - プルリクを出す側の工夫
- プルリクをレビューする側の工夫
工夫した内容

10. - プルリクが来たら３時間以内に返そう
- 基本的には今やってる業務より優先で見よう
- レビュー指摘は具体的に書こう
- 例）
これだと◯◯のパターンでクラッシュする可能性がありますね。
ここをこんな実装に変えるのはどうでしょう？
- レビューするのも人、レビューされるのも人、を意識しよう
- 思いやりのある言葉遣いを
プルリクをレビューする側の工夫

11. 工夫した結果

12. 工夫した結果
ただの参考値なので
休みとかで膨らんでも
OK

13. 爆速になりました！

14. https://hrmos.co/pages/sansan/jobs/1000208
https://meety.net/matches/gqCoZgvTdnDh
最後に