コードレビューはTLの最高レバレッジ活動の1つです。品質を保護し、教えることができます。目標はより良いコードとより強いチームであり、ゲートキーピングではありません。レビューは迅速で、親切で、重要な事に焦点を当てるべきです。
原則
text
✓ Review the CODE, never the person ("this function" not "you")
✓ Distinguish must-fix from nice-to-have (label nits explicitly)
✓ Ask questions, don't issue commands ("what about X?" invites discussion)
✓ Praise good work, not just problems
✓ Be fast — a PR blocked for two days kills momentum
✓ Approve when it's good enough, not perfect
具体例
「これは間違っています。マップを使ってください。」ではなく、「nit:ここでマップを使えばネストされたループを回避できます。どうですか?ブロックしていません。」と書きます。同じポイントですが、教えることができ、異なる意見の余地があり、マージをブロックしません。
パワーダイナミクスを監視する
TLとして、あなたのコメントは追加の重みを持っています。「私ならそうしない」というカジュアルな発言は命令のように感じられます。実際の懸念と好みの違いを明確にし、人々が異議を唱える余地を残してください。
