Administrator
发布于 2025-11-06 / 6 阅读
0
0

code reviewet 和 cl 二个关键字


🧩 一、Code Review 是什么?

Code Review(代码评审) 是指:

团队成员在代码提交(commit / merge)之前,由其他开发者对代码进行检查、审阅和反馈的过程。

✅ 主要目的:

  • 发现潜在的错误或安全隐患;

  • 保持代码风格一致;

  • 提升代码可维护性;

  • 帮助新人学习更好的实现方式。

📦 常见形式:

类型

说明

Formal Review(正式评审)

在重要版本合并前由指定评审人(Reviewer)系统检查。

Peer Review(同事互审)

团队成员之间相互审查日常提交。

Tool-Assisted Review(工具辅助)

通过GitHub、GitLab、Gerrit等平台自动发起review请求。


⚙️ 二、RT 是什么?

RT” 在不同环境下有多种含义,但在代码提交流程中最常见的两种解释如下:

场景

RT 含义

解释

Google / Android / 部分大厂内部

Review ThreadReview Tool

指代码评审系统本身(如Gerrit上的评审线程),例如“请在RT上回复评审意见”。

GitHub / GitLab 场景

Ready To Review / Ready To Test

表示代码已经准备好进入评审或测试阶段,例如开发者发起MR后在标题加上“[RT]”。

👉 举例:

“这个CL已经过自测,可以RT了”
意思是——该代码变更已完成开发并可提交进入评审(Review Thread)。


💡 三、CL 是什么?

CL 是 “Change List(变更集)” 的缩写。
它表示一次代码修改提交(可能包含多个文件的修改),是一个可独立审查的代码单元。

在 Gerrit、Perforce、Google 的开发体系中非常常见。

✅ 特征:

  • 一个 CL 对应一次独立提交;

  • 通常包含修改说明(Change Description);

  • 通过 Code Review 后才能被 合并(Submit / Merge)

  • 每个 CL 都有唯一的 ID(Change ID)。

👉 举例:

“请在 CL#10245 里修复那个NullPointer问题。”
表示在编号为10245的变更集中修正问题。


🧠 四、三者关系总结

名称

全称

含义

所处阶段

示例

CL

Change List

一次代码变更(commit)

提交阶段

CL#12345 修复登录崩溃

RT

Review Thread / Ready to Review

代码评审的线程或状态

审查阶段

该CL已RT,等待Reviewer审批

Code Review

Code Review

审查代码质量的过程

全流程

代码通过Review后可合并主分支

工作流程图示:

开发者提交修改(CL)
       ↓
发起评审(RT)
       ↓
进行 Code Review(Reviewer 审阅并反馈)
       ↓
通过 Review 后 Merge / Submit

🧭 五、延伸说明

在大型团队或使用 Gerrit、Perforce、Google 内部工具的环境中:

  • CL = 提交单位;

  • RT = 发起评审;

  • Code Review = 审核过程;

  • LGTM(Looks Good To Me) = 评审通过标志;

  • Submit/Merge = 合并到主干。


一句总结:

CL 是代码提交本身,RT 是发起评审的动作或线程,Code Review 是整个审查过程。


萍哥个人简介:http://quxiedu.com:6230/archives/ping-ge-ge-ren-jian-ji


评论