今日の英語表現 17/30 · ステップ: レビューを受ける・返信する 例文はすべてこの記事のために作成したものです。実際のりポジトリのコメントをそのまま持ってきたものではなく、コミットハッシュは例です。
目次
概要
| 表現 | Addressed in a1b2c3d. / I've addressed all the comments. |
| 意味 | a1b2c3d コミットで反映しました / すべての指摘を反映しました。 |
| 使う場面 | レビューのインラインコメントの返信、PR のコメント |
| 丁寧さ | 中立。報告する表現です。 |
1. 意味とニュアンス
address は「住所」という名詞だけでなく、「(問題や質問に)取り組む、対処する」という動詞としてもよく使われます。address a comment は「レビューの指摘を反映する」、address a concern は「懸念に対応する」という意味になります。fix は「直す」という意味なのでバグにしか合いませんが、address は質問に答えたり、ドキュメントを補足したり、設計を変更したりする作業まで幅広くカバーします。
Addressed in (コミット) は「どこで反映したか」を知らせてくれます。GitHub はコメント内のコミットハッシュを自動的にリンクに変換するため、レビュアーはそのリンクをクリックして該当の変更だけにアクセスできます。複数の指摘を反映した場合、インラインコメントごとにこのように返信しておくと、レビュアーが一つずつ確認して会話を Resolve（解決）できます。
コミットを分けて残すこととも関連しています。Kubernetes のコントリビューションガイドでは、レビュー中は修正内容を新しいコミットとしてアップロードし、まだスクワッシュしないように案内しています。レビュアーが修正分だけを個別に見られるようにするためです。スクワッシュはレビューの終盤に依頼を受けた際に行います（第21回）。スクワッシュや rebase を行うとハッシュが変わるため、そのあとでは Addressed in のリンクが古いコミットを指してしまう可能性がある点にも注意してください。
会話を誰が Resolve するかは、リポジトリやレビュアーの習慣によって異なります。レビュアーが自分で確認してクローズしたい場合が多いので、確信がない場合は作成者は返信だけを行い、クローズする作業はレビュアーに任せます。
2. 例文
状況 1: インラインコメントに返信する場合
Addressed in a1b2c3d. I switched to
errors.Is()as you suggested.
状況 2: 複数の指摘を一度に反映した場合
Thanks for the thorough review! I've addressed all comments in the last two commits:
- renamed
cfgtoconfig- added a nil check before dereferencing
spec.template- documented the new flag in the README
thorough は「綿密な、徹底的な」という意味です。レビュアーの労力をねぎらう言葉としてよく使われます。
状況 3: 一部だけを反映した場合
Addressed most of the comments in d4e5f6a. For the caching suggestion, I left a reply below; I'm not sure it's worth the added complexity here.
すべてを反映していない場合は、どの部分が残っているのかを明確に示します。
3. 似た表現との違い
- Done. / Fixed. 最も短い返答です。些細な指摘（nit）にはこれで十分です。
- Updated. 「修正した」という幅広い意味です。ドキュメントや説明を変更した際に自然です。
- Resolved. 会話自体が解決されたという意味で、GitHub の Resolve conversation ボタンと同じ言葉です。
4. 韓国人がよく犯すミス
- I reflected your comment. 「反映する」を reflect と直訳すると「鏡に映す」や「熟考する」のように受け取られてしまい、不自然です。
- 修正した文: I've addressed your comment. または I've applied your suggestion.
- I modified as you said. modify の目的語が抜けており、as you said はややお堅い表現です。
- 修正した文: Updated as suggested.
- All fixed. Please merge. レビュアーが確認する前にマージを依頼するのは早すぎます。
- 修正した文: I've addressed all the comments. PTAL.
5. 一行練習
次の韓国語の文をインラインコメントの返信に書き換えてください。
d4e5f6a コミットで反映しました。ご提案いただいた通り、タイムアウトを設定値に切り出しました。
模範解答
Addressed in d4e5f6a. I made the timeout configurable as you suggested.
「設定値に切り出す（設定可能にする）」は make ... configurable が最も自然です。extract to config と書いても意味は通じます。
Titi のひと言 「反映しました」のあとにコミットハッシュを1つ添えるだけでも、レビュアーのクリック数を減らすことができます。
参考
- Merriam-Webster, 「address」（動詞）: https://www.merriam-webster.com/dictionary/address
- Kubernetes コントリビューションガイド, Pull Request Process（レビュー中のスクワッシュ禁止）: https://github.com/kubernetes/community/blob/master/contributors/guide/pull-requests.md
- GitHub Docs, Autolinked references and URLs（コミットハッシュのリンク）: https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/autolinked-references-and-urls