コードレビューのミーティングを効果的に行うためのガイドライン。
時間を無駄にしないためのテクニックですね。
事前に読んでおく
レビュー対象物は事前に配布し、レビュアーは事前に読んでおき、チェックポイントはマークしておく。
一人でできる作業は事前に済ませておいて、みんなで集まっている時間をムダにしないようにする。
チェックリストの有効活用
すでにあるチェックリストの項目番号などでマークしておく。
個別の課題ごとに、どういう点が問題かの説明をはじめるのは時間のムダ。
すでに検討済みのチェックリストを最大限利用しましょう。
チェックリストがあっても、人間だから、ミスや見落としがあって、コードに同じ問題がでてくる。
レビューは検出だけ
前に書いたとおり。
修正方法の議論や実際の修正作業は別セッションにしたほうが効率的です。
Do one thing, do it well.
アクションリスト
検出結果は、アクションリストとして作成する。
コードの印刷物の蛍光ペンのマーキングでも十分です。
ただ、個数は確認しておきましょうね。
フォローアップミーティング
アクションリストの実行結果をもう一回レビューする。
フォローアップミーティングを必ず設定する。
ツールの活用
コードとモデルの同期とか、修正箇所の一覧作成とかはツールの機能を使いこなせば、自動化できます。
使える機能は積極的に活用して、みんなで楽をしましょう。
時間を無駄にしないためのテクニックですね。
事前に読んでおく
レビュー対象物は事前に配布し、レビュアーは事前に読んでおき、チェックポイントはマークしておく。
一人でできる作業は事前に済ませておいて、みんなで集まっている時間をムダにしないようにする。
チェックリストの有効活用
すでにあるチェックリストの項目番号などでマークしておく。
個別の課題ごとに、どういう点が問題かの説明をはじめるのは時間のムダ。
すでに検討済みのチェックリストを最大限利用しましょう。
チェックリストがあっても、人間だから、ミスや見落としがあって、コードに同じ問題がでてくる。
レビューは検出だけ
前に書いたとおり。
修正方法の議論や実際の修正作業は別セッションにしたほうが効率的です。
Do one thing, do it well.
アクションリスト
検出結果は、アクションリストとして作成する。
コードの印刷物の蛍光ペンのマーキングでも十分です。
ただ、個数は確認しておきましょうね。
フォローアップミーティング
アクションリストの実行結果をもう一回レビューする。
フォローアップミーティングを必ず設定する。
ツールの活用
コードとモデルの同期とか、修正箇所の一覧作成とかはツールの機能を使いこなせば、自動化できます。
使える機能は積極的に活用して、みんなで楽をしましょう。