このページでわかること
PRレビューコメント対応を、指摘確認、修正、差分確認、報告の作業例として見られます。
作業例の概要
PRレビュー対応では、指摘を無視せず、修正が必要なもの、質問だけのもの、確認待ちのものに分けてから作業します。
作業前に確認すること
PR、未解決コメント、対象branch、変更範囲、Secrets混入リスク、確認項目を先に見ます。
Codexに任せたこと
コメント分類、修正候補、対象ファイルの小さな変更、差分要約、返信文の下書きを任せます。
人間が確認したこと
review意図を誤読していないか、対象外変更がないか、CIや表示確認が必要かを確認します。
触らなかったもの
危険な履歴操作、Secrets、認証情報、DB、cron、DNS、API/OAuth、広告タグ、Search Consoleタグは触りません。
作業後に確認したこと
差分、内部リンク、CI結果、表示確認、未解決コメント、報告書、返信文を確認します。
失敗しやすい点
レビューを無視する、無関係な大改造を混ぜる、テストを無効化する、危険操作を推奨することは避けます。
FAQ
レビューコメントは全部修正しますか？
修正、質問、確認待ちに分けます。すべてがコード変更とは限りません。
Codexに返信文を書かせてもよいですか？
下書きは任せやすいですが、最終文面は人間が確認します。
CIが落ちたらどうしますか？
ログを確認し、原因候補と修正範囲を分けます。
無関係な変更も一緒に直しますか？
避けます。レビュー対象の範囲に絞ります。