LGTM 是什么
LGTM 是 Looks Good To Me 的缩写，评审者用它表示未发现阻塞性问题。它是一句表态，不是质量担保：CI 是否通过、分支保护要求的批准数是否满足、作者是否完成自查，仍然是合并的前置条件。给出 LGTM 之后代码出问题，责任仍在作者与合并者一侧。
基本信息
| 含义 | Looks Good To Me：未发现阻塞性问题 |
|---|---|
| 边界 | 不担保零缺陷；合并仍需 CI 通过与规则满足 |
| 有效期 | 只覆盖评审时的那个版本，作者推送新提交后需重新确认 |
| 常见变体 | LGTM with nits、LGTM after 修复 X |
| 平台表现 | GitHub 上通常对应 Approve，计入所需批准数 |
使用要点
- 可以附带 nit 或小建议：说明不阻塞合并，作者择机处理。
- 带条件的批准要写明前置条件，避免出现「批准了但仍有未决问题」。
- 只看了部分文件就表态时说明范围，别让作者误以为全量看过。
- 作者推送新提交后，旧批准未必覆盖新代码，重要改动应重新过一遍。
- 批准不代替验证：关键改动仍需作者给出可核对的测试或复现结果。
表态写法与含义
| 写法 | 含义 | 对合并的影响 |
|---|---|---|
| LGTM | 无阻塞意见 | 满足其他条件即可合并 |
| LGTM with nits | 有非阻塞建议 | 可合并，建议择机处理 |
| LGTM after 修复 X | 带前置条件 | X 修好并确认后才能合并 |
| 评论但不表态 | 未给结论 | 不计入批准数 |
表态前的自检
- 确认自己看过全部改动文件，或者明确说明只看了哪一部分。
- 看到测试缺失或验证方式含糊时先提问，不要用 LGTM 盖章。
- 作者继续推送新提交后重新看增量，而不是默认沿用旧批准。
- 在分支保护下 Approve 会直接解锁合并按钮：点之前再看一次 CI 状态。
LGTM 在 Google 评审文化与开源社区中被广泛使用，语义接近「我没有阻塞意见」，而不是「我保证没问题」。
批准也不是流程终点：合并后的监控项与回滚预案仍由作者准备。
收到 LGTM 后作者仍要做最后一遍自审：批准的是当时那一版 diff，最后一刻的改动不会被自动覆盖。