サブタスクなコードレビュー考察
他チームのコードレビューを依頼された場合の考察。
他チームのコードレビュー依頼がある状況と対応方法
該当チームのリードエンジニアのリソースが足りていない
一時的なリソース貸出に当たると思われるので、期日やレビュー範囲などの合意が取れればOKだと考える。
該当チームにリードエンジニア( が存在しない || の実力不足 )
ある一定以上の品質達成が該当チームだけでは成し得ない状況だと思われる。
この状態でのレビューは想像以上に高いコストとなることもあり、
最悪、何度も発生する差し戻しなどで、プロジェクト全体の進捗が滞ってしまう事もありえる。
そうならないように、品質的にはNGでもレビューOKとするルールを設定する必要があると考る。
例えば、以下のルールを設定する。
- 一定の差し戻し回数を超えたものに関してはレビューOKとする
- レビューは行うが、修正は行わないものとする
- 今後の開発時に意識してほしい観点を伝えることを目的としたレビュー
まとめ
他チームのコードレビューは、レビューに関するルールの合意をとって実施するべき。