Code Review, 指代码审阅;场景:新人或者初级工程师,做了一些改动,但是需要在老程序员的确认后再添加至主分支或特性分支;不可能让老程序员坐你旁边看代码吗,这样太浪费时间了; 举个例子:A项目有两个分支master分支和dev分支;现在公司在开发,所有人完成任务,都应该把分支合并至dev分支; 张三在自己的本地开发环境,基于dev分支,创建zhang_san分支;花了三天,完成一个功能; 老程序员觉得张三是新手,不放心就这么将他的改动直接放进dev分支;就让张三提个Merge申请,让他Code Review; 于是张三就将自己的zhang_san分支提到远程服务器上,操作“New Merge Request”,发起一个Merge请求,将zhang_san合并至dev分支,指定审阅人是老程序员; 老程序员找了个空闲时间,查看这个Merge Request,觉得有点问题, 于是就提了一些建议(提的建议会自动发送到相关人的关联邮箱); 张三后来通过邮箱查看到老程序员提的建议,在自己本地zhang_san进行了修改,并git push至远程zhang_san分支; 这时的“Merge Request”会更新最新的Commit;老程序员再看这次总的改动,发现不错,挺好的,点击确认请求后,就成功将zhang_san分支上的改动合并至dev分支啦;【到这就完成了Code Review】 张三通过邮件,知道自己的分支已经被合并进开发分支dev很开心;可以删除这个本地zhang_san分支和远程zhang_san分支,保持分支干净;也可以继续在zhang_san分支做新的开发,提交新的Merge_Request,进行新一轮的Code Review。
Code Review
, 指代码审阅;场景:新人或者初级工程师,做了一些改动,但是需要在老程序员的确认后再添加至主分支或特性分支;不可能让老程序员坐你旁边看代码吗,这样太浪费时间了;举个例子:
A项目有两个分支
master
分支和dev
分支;现在公司在开发,所有人完成任务,都应该把分支合并至dev
分支;张三在自己的本地开发环境,基于
dev
分支,创建zhang_san
分支;花了三天,完成一个功能;老程序员觉得张三是新手,不放心就这么将他的改动直接放进
dev
分支;就让张三提个Merge
申请,让他Code Review
;于是张三就将自己的
zhang_san
分支提到远程服务器上,操作“New Merge Request”,发起一个Merge
请求,将zhang_san
合并至dev
分支,指定审阅人是老程序员;老程序员找了个空闲时间,查看这个
Merge Request
,觉得有点问题, 于是就提了一些建议(提的建议会自动发送到相关人的关联邮箱);张三后来通过邮箱查看到老程序员提的建议,在自己本地
zhang_san
进行了修改,并git push
至远程zhang_san
分支;这时的“Merge Request”会更新最新的
Commit
;老程序员再看这次总的改动,发现不错,挺好的,点击确认请求后,就成功将zhang_san
分支上的改动合并至dev
分支啦;【到这就完成了Code Review】张三通过邮件,知道自己的分支已经被合并进开发分支
dev
很开心;可以删除这个本地zhang_san
分支和远程zhang_san
分支,保持分支干净;也可以继续在zhang_san
分支做新的开发,提交新的Merge_Request
,进行新一轮的Code Review
。