今日英语表达 15/30 · 阶段：接收审阅与回复。 示例句子全为本文虚构。并非摘自实际仓库的评论，人名和编号均为示例。
目录
一目了然
| 表达 | LGTM / LGTM with one nit / /lgtm |
| 含义 | 看起来不错（批准） / 除了一个小问题外都很不错 |
| 使用场景 | 审阅结论、Prow 命令 |
| 礼貌程度 | 中立。用于传达审阅结论 |
1. 含义与语境
LGTM 是 Looks Good To Me 的缩写。LLVM 词汇表将 LGTM 解释为「审阅者认为该补丁可以提交」，LLVM 代码审阅文档则记载代码提交的批准通常通过包含 LGTM 的消息来表示。同一文档建议信心不足的审阅者做出有条件的批准，如「LGTM, but please wait for (其他审阅者)」。
注意 to me 这个词。既然是「对我来说看起来不错」，那么其他审阅者或维护者的判断可能仍然存在。因此，单次 LGTM 并不直接等于合并。
Kubernetes 与 Prow 的 /lgtm、/approve。 在 Kubernetes 系列仓库中，如果在评论中输入命令，Prow 机器人就会添加标签。根据 Kubernetes 的 OWNERS 文档，说明如下：
/lgtm: 表示审阅者在查看了代码质量、正确性和风格后认为不错。机器人会添加lgtm标签。GitHub 组织成员可以成为审阅者，但 PR 作者不能在自己的 PR 上加 /lgtm。 取消是/lgtm cancel。/approve: 表示批准者已确认依赖项、兼容性、API 和标志定义等更广泛的批准标准。只有在对应目录的 OWNERS 文件中有名单的人才能使用，当收集到所有必需的 OWNERS 批准时，会附上approved标签。- 合并自动化通常在同时具备
lgtm和approved两个标签，且没有诸如do-not-merge/hold这样阻止合并的标签时进行合并。
因此，在 Kubernetes 系列仓库中，“收到 LGTM”与“可以合并”是两码事。可能还需要另外获得批准者的 /approve。
2. 示例句子
情况 1：审阅者的有条件批准
LGTM with one nit. Feel free to merge after fixing the typo in the comment.
情况 2：在使用 Prow 的仓库中
Thanks for the quick turnaround. The changes look good to me.
/lgtm
quick turnaround 意为“快速响应”，是对迅速处理审阅意见的作者的赞扬。
情况 3：作者收到 LGTM 后
Thanks for the review! I've fixed the nit. Do I need an approval from someone in OWNERS, or is this good to go?
good to go 是口语，意为“可以开始了”或“没问题了”。
3. 相似表达与区别
- Approve（GitHub 审阅功能）。 GitHub 的 Approve 按钮根据仓库设置有时会成为合并条件。它与 Prow 的 /approve 是不同的机制。
- Looks good overall, but ... 意为“总体不错，但……”，表示尚未批准，还有需要修改的地方。
- SGTM (Sounds Good To Me)。 当同意某个提议或计划（而非代码）时使用。
4. 韩国人常犯的错误
- 作者在自己的 PR 上 加 /lgtm。 规则不允许，机器人会拒绝。
- 修改后的句子：Thanks! Could someone from OWNERS take a look for approval?
- Thank you very very much for your LGTM!! 感谢是好事，但在工作对话中过分强调和使用感叹号会显得很不自然。
- 修改后的句子：Thanks for the review!
- 一收到 LGTM 就 催促「Please merge」。 其他审阅者或批准者的判断可能尚未完成。
- 修改后的句子：Thanks! Is there anything else needed before this can be merged?
5. 行动练习
请以审阅者的身份在评论中写下以下内容。
整体很好。只需修复一个测试名称拼写错误即可。
参考答案
LGTM with one nit: there's a typo in the test name.
LGTM with one nit 这个表达中同时包含了“批准”和“一个微小修改”。冒号后面简短写出具体是哪一个修改。
小贴士。 LGTM 意为“对我来说不错”。距离合并还有哪些未完成的步骤，请查看仓库规则。
参考资料
- Kubernetes 贡献指南，OWNERS 文件 (/lgtm, /approve): https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md
- LLVM 词汇表，“LGTM”: https://llvm.org/docs/Lexicon.html
- LLVM 代码审阅政策与实践: https://llvm.org/docs/CodeReview.html