换个角度想
PR 像论文投稿:稿子写完不能直接见刊,审稿人提意见、你修改,通过了才发表。主线就是那本期刊,进去的都过了同行评审。
术语 · 工程与协作
Pull Request
请求团队审查一组代码修改,并在确认后合并到目标分支。
合并之前,先过一道评审
PR 像论文投稿:稿子写完不能直接见刊,审稿人提意见、你修改,通过了才发表。主线就是那本期刊,进去的都过了同行评审。
标题重构支付模块
内容完整 Diff + 改动说明
没有闸门
改动已经躺在主线上了
测试✓ 通过
构建✓ 通过
第一个「评审」是线上用户
老账户的余额显示成了 0
第 87 行「老账户没这个字段吧?」
你补上兼容逻辑 ✓
分支改完,发起 PR
Agent 写的代码先出 PR,人看过 Diff 再进主线。
评审里的讨论,让整个团队理解这次改动。
半年后回看,当时为什么这么改,PR 里全有。
深夜赶工,AI 生成了一个大改动,测试也绿了,能不能跳过 PR 直接推主线?
测试绿只说明没破坏已有用例,用例覆盖不到的坑机器看不见。越是大改动、越是深夜,越需要第二双眼睛——发 PR,明早让同事看过再合。
PR 评审不是走形式。「LGTM 秒批」的团队,主线和没有评审时一样脆——评审的价值全在认真看 Diff。