こんにちは、freee でエンジニアをしている @nasa4w です。 この記事は freee Engineers Advent Calendar の10日目です。 今回は大きな機能を複数のPRに分割する話をします。 freeeでも絶賛模索中の取り組みなので現在進行形の話ですm(_ _)m まとめ - 今回作ったブランチとPR - 長くなってしまったので結論をさきに置いておきます。PRを複数に分割する場合のゴールはこんな感じ。 Why?(なぜこうしたか) こうすることで、常に work ブランチを使って動作確認ができる レビューする PR を2つに分割できる view の PR と logic の PR レビュー指摘の反映は、topic_view, topic_logic ブランチに commit すればOK work ブランチにはこの2つのブランチを merge しなおすだけでOK PR
![レビューしやすいPRの作り方 - 大きいPRを複数に分割する - - Qiita](https://cdn-ak-scissors.b.st-hatena.com/image/square/4913a7c154f7575c9bac3f5852fb6be9b7a9d147/height=288;version=1;width=512/https%3A%2F%2Fqiita-user-contents.imgix.net%2Fhttps%253A%252F%252Fcdn.qiita.com%252Fassets%252Fpublic%252Fadvent-calendar-ogp-background-f625e957b80c4bd8dd47b724be996090.jpg%3Fixlib%3Drb-4.0.0%26w%3D1200%26mark64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTkxNiZoPTMzNiZ0eHQ9JUUzJTgzJUFDJUUzJTgzJTkzJUUzJTgzJUE1JUUzJTgzJUJDJUUzJTgxJTk3JUUzJTgyJTg0JUUzJTgxJTk5JUUzJTgxJTg0UFIlRTMlODElQUUlRTQlQkQlOUMlRTMlODIlOEElRTYlOTYlQjklMjAtJTIwJUU1JUE0JUE3JUUzJTgxJThEJUUzJTgxJTg0UFIlRTMlODIlOTIlRTglQTQlODclRTYlOTUlQjAlRTMlODElQUIlRTUlODglODYlRTUlODklQjIlRTMlODElOTklRTMlODIlOEIlMjAtJnR4dC1jb2xvcj0lMjMzQTNDM0MmdHh0LWZvbnQ9SGlyYWdpbm8lMjBTYW5zJTIwVzYmdHh0LXNpemU9NTYmdHh0LWNsaXA9ZWxsaXBzaXMmdHh0LWFsaWduPWxlZnQlMkNtaWRkbGUmcz05YmY2ZjdmODJiZjBmOWQ3N2Y0MTZjMmNmNWMzZjhmNg%26mark-x%3D142%26mark-y%3D151%26blend64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTYxNiZ0eHQ9JTQwbmFzYTR3JnR4dC1jb2xvcj0lMjMzQTNDM0MmdHh0LWZvbnQ9SGlyYWdpbm8lMjBTYW5zJTIwVzYmdHh0LXNpemU9MzYmdHh0LWFsaWduPWxlZnQlMkN0b3Amcz03MWNmNDA2YTE5NWQ1MTUzMzY3OTEzN2EyZGVjNGIxOQ%26blend-x%3D142%26blend-y%3D491%26blend-mode%3Dnormal%26s%3Dd01fa87c5102a6b4f8a0b6e3c2734a33)