Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.
Upcoming SlideShare
What to Upload to SlideShare
What to Upload to SlideShare
Loading in …3
×
1 of 15

Sansan androidチームが取り組む コードレビューを早くまわす工夫

0

Share

Download to read offline

2021/11/07
DevFest GDG Shikoku

Related Books

Free with a 30 day trial from Scribd

See all

Sansan androidチームが取り組む コードレビューを早くまわす工夫

  1. 1. Sansan Androidチームが取り組む コードレビューを早くまわす工夫 Sansan株式会社 技術本部 Mobile Application Group 古川 真次 DevFest Shikoku 2021
  2. 2. 自己紹介 ● 古川 真次(ふるしん) ●  @furusin_oriver ● Sansan,Inc. Android Engineer, Team Leader ● 大阪府在住 ● 趣味:勉強会の主催 ○ GDG Osaka オーガナイザー ○ Kansai.kt オーガナイザー
  3. 3. Sansan Androidチームが取り組む コードレビューを早くまわす工夫 Sansan株式会社 技術本部 Mobile Application Group 古川 真次 DevFest Shikoku 2021
  4. 4. 工夫する前 - レビューが終わらないと後続の実装をプルリク出しにくい - レビューが終わってないから後続を実装しにくい - いつ頃レビュー終わりますかね… - 24時間以内にはレビュー返そうねってルールにはなってた
  5. 5. - プルリクを出す側の工夫 - プルリクをレビューする側の工夫 工夫した内容
  6. 6. - プルリクを出す側の工夫 - プルリクをレビューする側の工夫 工夫した内容
  7. 7. - プルリクのサイズはできれば小さくしよう - 最大でも200行までにしよう - Descriptionを充実させよう - 動作確認してるはずなので「動作確認した内容」をしっかり書こう - 「正しく動くこと」は絶対に使うな - 「正しい」とは?ってなる - 「◯◯を押したら□□となること」と書こう プルリクを出す側の工夫
  8. 8. Descriptionを充実させよう
  9. 9. - プルリクを出す側の工夫 - プルリクをレビューする側の工夫 工夫した内容
  10. 10. - プルリクが来たら3時間以内に返そう - 基本的には今やってる業務より優先で見よう - レビュー指摘は具体的に書こう - 例) これだと◯◯のパターンでクラッシュする可能性がありますね。 ここをこんな実装に変えるのはどうでしょう? - レビューするのも人、レビューされるのも人、を意識しよう - 思いやりのある言葉遣いを プルリクをレビューする側の工夫
  11. 11. 工夫した結果
  12. 12. 工夫した結果 ただの参考値なので 休みとかで膨らんでも OK
  13. 13. 爆速になりました!
  14. 14. https://hrmos.co/pages/sansan/jobs/1000208 https://meety.net/matches/gqCoZgvTdnDh 最後に

×