PR に範囲外の変更が混ざっていないか確認する
収録 2026-11-01
ついでの変更を洗い出し、レビューが無関係な差分に時間を取られないようにします。
PR の説明と変更ファイル一覧を渡します。1)目的に必要な変更と「ついで」の変更を分ける;2)ついでの変更のうち挙動に影響するもの;3)各変更の扱い(分割・注記して残す・戻す);4)レビューで飛ばされやすいが最も危険な箇所;5)そのまま貼れるコメント一文。コードスタイルは評価しない。
当サイトオリジナル
中国語の原文
使い方
评审 PR 时先看「和这个目标无关的文件」。
こう変えてみる
- コードを書く前に要件と制約を復唱させる。
- 全体書き換えではなく、最小の変更とロールバック方法を出させる。
得られる成果物
- そのまま使える最小変更(コピー可能なコード)
- 影響範囲:変更ファイルと追従が必要な呼び出し元
よくある失敗
- 一度に十ファイル変更させ、何が原因か分からなくなる。
- エラーだけ貼ってコードとバージョンを省くと、使えない修正が返る。
出力のチェック
- 変更は動いて戻せて、無関係なコードに手を入れていない。
- 境界条件(空・巨大・同時実行)が説明かテストで押さえられている。
使いどころと注意
- 向いている用途:要件と制約を復唱させてから、最小限の動く変更を出させる。
- 注意:エラーにはバージョンと最小再現を添える。無いと一見もっともらしいが合わない修正が返る。