TT Lab
博客

한국어 English 日本語 中文

开始
博客

博客

Addressed in a1b2c3d: 告知在哪个提交中修复了评审指出的问题

今日英语表达 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 cfg to config
  • 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. 相似表达与区别

4. 韩国人常犯的错误

5. 一行练习

请将以下韩语句子改写为行内评论的回复。

已在 d4e5f6a 提交中修改。按照您的建议,我将超时时间提取为了配置项。

参考答案

Addressed in d4e5f6a. I made the timeout configurable as you suggested.

“设为配置项”用 make ... configurable 最为自然。使用 extract to config 也能表达清楚意思。

Titi 的小贴士。 在“已修改”后面只需附上一个提交哈希,就能减少评审人员的点击次数。

参考资料

登录后即可点赞

评论

还没有评论。

登录后即可发表评论