次のいずれかであるため、スプリントのすべての開発者にとって最優先のアクティビティとしてコードレビューを説明する 思考の学校 があります。
a)未実現価値の源泉
b)バグの原因
白黒表示などを購読するかどうかはわかりませんが、その意見には同意します。
しかし、背景は十分です。
レビューが一度に何日も待ち行列に残るという永続的な問題があります。ボトルネックは常に需要が高いため、主要な開発者が原因です。また、彼らの仕事の重要な性質のために、彼らはプログラムをペアにする傾向があるので、彼ら自身のコードレビューをそのように行う必要はありません。したがって、彼らが本当の意味でレビューを行うための真のインセンティブはありません。
最も頻繁に使用される不満は、プログラミング中に思考の列を失うことです。これにはいくつかの真実がありますが、確かに彼らは昼食後や朝の最初のことだけをレビューできますか?
これを回避する方法はありますか?以前のプロジェクトでは、1日のSLAを適用しましたが、これはほとんどの部分でうまく機能しましたが、ペアのプログラマーによってルールが破られる傾向がありました。
開発者は順番にコードレビューを行うので、負荷はすでにかなり共有されています。
チームには、自分のコードをレビューする権限が必要です。結局、資格のある人がいないからです。
GITプルリクエストは、このための最も簡単で最も堅牢なモデルの1つを実装します。
簡単にするためにできることがいくつかあります。
ペア(または暴徒)プログラミングを実行する場合、ピアレビューは必要ですか?
私は ピアコードレビューを実行する理由についての回答を以前に記述しました を最初に使用しました。ただし、これらはすべてペアプログラミングでも対処できます。本質的には、ペアプログラミングは、リアルタイムで高度にインタラクティブなコードレビューです。
コードはペアでプログラムする(そして効果的なペアリング手法に関するチームの基準やガイドラインを設定する)か、コードレビューを行う必要があると言う方が簡単かもしれませんが、どちらも必須ではありません。もちろん、いつでもコードレビューを追加して変更の3番目(または4番目)の目を得ることができますが、2人のユーザーがすべての変更の品質を確認していることを確認する必要があるだけの場合、ペアプログラミングはあなたのその。