良いレビューは、コードだけでなくチームの信頼を育てます。逆に、棘のある指摘は生産性を静かに蝕みます。
「人」ではなく「コード」を対象にする
「なぜこう書いたの?」より「ここはこうするとこういう利点があります」。主語をコードに置くだけで、指摘は攻撃でなく提案になります。
指摘の重要度を明示する
- [must] 直してほしい(バグ・セキュリティ)
- [imo] できれば(設計の好み)
- [nits] 些細(タイポ等)
ラベルを付けるだけで、相手は優先度を判断でき、無用な議論が減ります。
良い点も書く
指摘だけのレビューは疲弊を生みます。「この抽象化は分かりやすい」など、良い実装を言語化して共有しましょう。
議論が長引いたら同期に切り替える
3往復しても平行線なら、テキストをやめて短い通話へ。時間を溶かさないのも技術です。