今日英语表达 17/30 · 阶段:接收并回复评审。 所有例句皆为本文虚构。并非摘自实际仓库的评论,提交哈希也仅为示例。
目录
一目了然
| 表达 | Addressed in a1b2c3d. / I've addressed all the comments. |
| 含义 | 已在 a1b2c3d 提交中处理 / 已处理所有评审意见。 |
| 使用场景 | 行内评论回复、PR 评论 |
| 礼貌程度 | 中性。这是一种汇报性质的表述。 |
1. 含义与语境
address 除了作名词表示“地址”外,也经常作为动词表示“处理、应对(问题或提问)”。address a comment 意为“采纳评审意见”,address a concern 意为“回应顾虑”。fix 意为“修复”,因此只适合用于 bug,而 address 的范围更广,甚至可以延伸到回答问题、补充文档或更改设计。
Addressed in (提交) 可以告知“在哪里进行了修改”。GitHub 会自动将评论中的提交哈希转换为链接,因此评审人员只需点击该链接即可查看相关更改。如果采纳了多条意见,在每条行内评论下方都这样回复,评审人员就可以逐一确认并 Resolve 对话。
这还与单独保留提交有关。Kubernetes 贡献指南建议,在评审期间应将修改作为新提交上传,暂时不要 squash。这是为了让评审人员能够单独查看修改部分。通常在评审接近尾声且收到请求时才会进行 squash(第 21 篇)。需要注意的是,进行 squash 或 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 的小贴士。 在“已修改”后面只需附上一个提交哈希,就能减少评审人员的点击次数。
参考资料
- Merriam-Webster, “address”(动词): https://www.merriam-webster.com/dictionary/address
- Kubernetes 贡献指南,Pull Request Process(评审期间不进行 squash): 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