ふだんの開発で小さなPRに分けるために考えていること - 私が歌川です

ふだんの開発でPRを出すときに考えていること - 私が歌川です の続編です。小さなPRを出す、という話がありましたが、どうやって小さくしているのか、というのを書きます。 どういうメソッドが欲しいか考える Aという情報をBという画面に出したい Aを取得するメソッドがないので、それを作る必要がある (メソッドCとする) というときに、メソッドCを作るPRをまず先に出して、マージされたら続いてCを使って得られる情報Aを画面Bに出すPRを出す、というのをやります。 実装方針を考えて既存実装を眺めてからそういう作戦を立てることもあれば、実装しているうちにメソッドCが足りないな、と気づいて別ブランチでPRを作ってそちらを先に出す、ということもあります。 このとき、1つのPRで両方やろうとすると、メソッドCの実装のレビューと、情報Aを画面Bに出す実装のレビューが同じPRで行われることになります。メソッド

[rkosaka]

だいたい自分も同じこと考えてる。作業時間が限られていると効果を感じる。

[Sixeight]

いい話。同様にコミットも小さな変更のみにしたい。

[yujiorama]

リファクタリングしてPR、ロジックを作ってPR、プレゼンテーションを作ってPR、ぐっとこらえて分ける。

[hitode909]

いい話