Closed2
PRの粒度
seiya2130参考
seiya2130まとめ
1つのことだけに注力する
作成するPRが1つのことだけを行なっているかどうかで変更行数・ファイル数は関係ない
(粒度が大きすぎるとレビューの負荷が高まり品質の低下につながるため)
粒度の目安として、
- revert単位として適切か
- レビュワーのコンテキストスイッチは最小限に収まっているか
- 複数の対応を一度に入れ込んでないか
以下の場合、粒度が適切なのは前者
- 特定の関数名を一括置換して1万行変更
- 画面開発とリファクタで20行変更
そのためにタスク分解が必要になる
タスク分解
マークダウンのチェックボックスでOK
大枠から分解していくことを意識して作成する
小さい機能追加を何回も繰り返して結果的に大きな機能を完成させるイメージを持つ
この時メンバーのレビューも入れると意識が揃って良い
途中でさらに分解するのもOK
目安は200-400行
200−400行は1時間以内にレビューが完了できるレベル
1時間以内に完了するレビューは不具合の発見率が高い
このスクラップは7時間前にクローズされました