こんにちは、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/aac4c3bc066a0e0c69449f3245ba25d1b89724ac/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-7940cd1c8db80a7ec40711d90f43539e.jpg%3Fixlib%3Drb-4.0.0%26w%3D1200%26mark64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTk3MiZoPTM3OCZ0eHQ9JUUzJTgzJUFDJUUzJTgzJTkzJUUzJTgzJUE1JUUzJTgzJUJDJUUzJTgxJTk3JUUzJTgyJTg0JUUzJTgxJTk5JUUzJTgxJTg0UFIlRTMlODElQUUlRTQlQkQlOUMlRTMlODIlOEElRTYlOTYlQjklMjAtJTIwJUU1JUE0JUE3JUUzJTgxJThEJUUzJTgxJTg0UFIlRTMlODIlOTIlRTglQTQlODclRTYlOTUlQjAlRTMlODElQUIlRTUlODglODYlRTUlODklQjIlRTMlODElOTklRTMlODIlOEIlMjAtJnR4dC1hbGlnbj1sZWZ0JTJDdG9wJnR4dC1jb2xvcj0lMjMzQTNDM0MmdHh0LWZvbnQ9SGlyYWdpbm8lMjBTYW5zJTIwVzYmdHh0LXNpemU9NTYmcz02NzgzNmY2YjNmMGNmOTRhODk3OGE4YTk1NDMyNjBkOA%26mark-x%3D120%26mark-y%3D96%26blend64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZoPTc2Jnc9OTcyJnR4dD0lNDBuYXNhNHcmdHh0LWNvbG9yPSUyMzNBM0MzQyZ0eHQtZm9udD1IaXJhZ2lubyUyMFNhbnMlMjBXNiZ0eHQtc2l6ZT0zNiZ0eHQtYWxpZ249bGVmdCUyQ3RvcCZzPWE5YjljOTM2Nzk4NGMwNTg4N2EwMzlhNDdjMGZlMWYw%26blend-x%3D120%26blend-y%3D500%26blend-mode%3Dnormal%26s%3D867d752d0bd468edad060a7fff04c5b3)