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

リーダブル プルリクエスト 分割プルリクエスト編 / Readable Pull Request

リーダブル プルリクエスト 分割プルリクエスト編 / Readable Pull Request

「コードレビューが順調に進んだほうが、チームの生産性が上がる可能性が高い」ということについては、違和感なく思う人が多いのではないでしょうか。

「順調に進める」ためには、それを意識することだけでなく、レビューする側・レビューを依頼する側のそれぞれにテクニックが必要になってきます。

このトークでは、かつて、「おまえのプルリクエストは大きすぎてレビューできない(意訳)」と実際に言われたことがある発表者が、現在までにどのような工夫をして「分割プルリクエスト」にたどり着いているのかを共有したいと思います。

なお、このトークのタイトルは「リーダブルコード」にインスパイアされていますが、トーク内容については特に関係するものではありません。

■想定する対象者

プルリクエストのサイズが大きくなりがちなかた
■話さないこと

コード自体をどう読みやすく書くか、など

Norio Suzuki

October 08, 2023
Tweet

More Decks by Norio Suzuki

Other Decks in Technology

Transcript

  1. ϓϧϦΫΤετͷঢ়ଶ $POUSPMMFS $service->process($id); 4FSWJDF public function process(int $id, string $type):

    void { // ... } ϓϧϦΫΤετ ύϥϝʔλෆ଍Ͱ ςετ΍੩తղੳ͕ ࣦഊ͢Δ
  2. ϓϧϦΫΤετͷঢ়ଶ $POUSPMMFS $service->process($id); 4FSWJDF public function process(int $id, string $type

    = "TYPE_A"): void { // ... } ϓϧϦΫΤετ σϑΥϧτ஋Λઃఆ͢Δͱɺ $POUSPMMFSͷςετΛ ௨͢͜ͱ͕Ͱ͖Δ
  3. ϓϧϦΫΤετͷঢ়ଶ $POUSPMMFS $service->process($id, $type); 4FSWJDF public function process(int $id, string

    $type): void { // ... } ৽نύϥϝʔλΛҾ͖౉͠ ຊདྷ૝ఆͨ͠ίʔυͷঢ়ଶʹ ϓϧϦΫΤετ ෆཁͳΒσϑΥϧτ஋΋࡟আ
  4. ࣦഊஊ $POUSPMMFS" $service->process($id, $type); 4FSWJDF public function process(int $id, string

    $type = "TYPE_A"): void { // ... } $POUSPMMFS# $service->process($id); ύϥϝʔλͷ௥Ճ࿙Ε͕͋ͬͯ΋ ੩తղੳͰ͸ؾ͕෇͔ͳ͍
  5. -JOLT w *ODSFNFOUBM13T w IUUQTXXXHBUTCZKTDPNCMPHHJUXPSL fl PXT w 1VMM3FRVFTUΛখ͘͢͞Δઓུ։ൃνʔϜͷύϑΥʔϚϯε޲্ͷͨΊͷୈҰา w

    IUUQTBHJMFKPVSOFZV[BCBTFDPNFOUSZ w ద੾ͳ໰୊ઃఆͱখ͘͞ϦϦʔε͢Δͱ͍͏͜ͱ w IUUQTTQFBLFSEFDLDPNTPVEBJSFMFBTFTNBMM