0/8
🎬 开发提交了一个改动较大的 PR 请评审人看:两人先对齐说明,再讨论一处设计取舍,评审人给出一条不阻塞合并的小建议,最后通过。
开发工程师
Could you review my pull request today?
你今天能评审一下我的 PR 吗?
请求评审最常用的礼貌句式。
评审人
Sure. The diff is large, can you add a summary in the description?
可以。改动比较大,能在描述里补一段说明吗?
评审前要求说明,是很多团队的规则。
开发工程师
Done. The first commit only renames files, the second one adds the retry logic.
已补。第一个提交只是重命名文件,第二个加了重试逻辑。
把大改动拆开讲,评审人更容易看。
评审人
Thanks, that helps. Why did you choose exponential backoff?
谢谢,这样清楚多了。为什么选指数退避?
提问针对设计取舍,而不是风格偏好。
开发工程师
Because the vendor API rate limits us at 100 requests per minute.
因为供应商接口限流每分钟 100 次。
回答要给依据,不能只说 I think so。
评审人
Makes sense. Nit: this log line is longer than the linter limit.
有道理。小意见:这行日志超过了 linter 的长度限制。
Nit 表示小建议,不阻塞合并。
开发工程师
Fixed. Anything else before I merge?
已修。合并前还有别的意见吗?
合并前主动确认,避免来回。
评审人
No, it looks good to me. Approved.
没有了,我看没问题,通过。
Approved 是评审通过的常用说法。
🎬 同场景其它对话
📚 继续学习
❓ 常见问题
这组对话该怎么练?
先用「整段连播」听一遍熟悉节奏,再切到「角色扮演」,选 开发工程师 或 评审人 自己说一遍,最后对照标准说法修正用词和语气。
句子下面的小字说明是做什么的?
说明这句话为什么这么说:时态、固定搭配或者语气。比单纯背句子更有用,换个场景也能套用。
能不能只听不看?
可以。用「整段连播」时当前句会自动高亮并滚动到可见位置,可以放到一边跟着听。