84669 人学习
152542 人学习
20005 人学习
5487 人学习
7821 人学习
359900 人学习
3350 人学习
180660 人学习
48569 人学习
18603 人学习
40936 人学习
1549 人学习
1183 人学习
32909 人学习
新手刚入门,能否简单讲一下code review是什么东西?怎么进行code review?谢谢
认证高级PHP讲师
Code Review, 指代码审阅;场景:新人或者初级工程师,做了一些改动,但是需要在老程序员的确认后再添加至主分支或特性分支;不可能让老程序员坐你旁边看代码吗,这样太浪费时间了;
Code Review
举个例子:A项目有两个分支master分支和dev分支;现在公司在开发,所有人完成任务,都应该把分支合并至dev分支;
master
dev
张三在自己的本地开发环境,基于dev分支,创建zhang_san分支;花了三天,完成一个功能;
zhang_san
老程序员觉得张三是新手,不放心就这么将他的改动直接放进dev分支;就让张三提个Merge申请,让他dev分支;就让张三提个Merge申请,让他Code Review;
Merge
于是张三就将自己的zhang_san分支提到远程服务器上,操作“New Merge Request”,发起一个Merge请求,将zhang_san合并至dev分支,指定审阅人是老程序员;
老程序员找了个空闲时间,查看这个Merge Request,觉得有点问题, 于是就提了一些建议(提的建议会自动发送到相关人的关联邮箱);
Merge Request
张三后来通过邮箱查看到老程序员提的建议,在自己本地zhang_san进行了修改,并git push至远程zhang_san分支;
git push
这时的“Merge Request”会更新最新的Commit;老程序员再看这次总的改动,发现不错,挺好的,点击确认请求后,就成功将zhang_san分支上的改动合并至dev分支啦;【到这就完成了Code Review】
Commit
张三通过邮件,知道自己的分支已经被合并进开发分支dev很开心;可以删除这个本地zhang_san分支和远程zhang_san分支,保持分支干净;也可以继续在zhang_san分支做新的开发,提交新的Merge_Request,进行新一轮的dev很开心;可以删除这个本地zhang_san分支和远程zhang_san分支,保持分支干净;也可以继续在zhang_san分支做新的开发,提交新的Merge_Request,进行新一轮的Code Review。
Merge_Request
code review 就是代码审查,你可以百度搜索一下,有很多文章
你写的代码,别人看,别人的代码,你也看
review 过了,再上线
代码审查,一般大公司都需要code review。如果代码有问题或者不符合规范,修改正确后再上线。
要你命三千:老代码中的那些坑
Code Review
, 指代码审阅;场景:新人或者初级工程师,做了一些改动,但是需要在老程序员的确认后再添加至主分支或特性分支;不可能让老程序员坐你旁边看代码吗,这样太浪费时间了;举个例子:
A项目有两个分支
master
分支和dev
分支;现在公司在开发,所有人完成任务,都应该把分支合并至dev
分支;张三在自己的本地开发环境,基于
dev
分支,创建zhang_san
分支;花了三天,完成一个功能;老程序员觉得张三是新手,不放心就这么将他的改动直接放进
dev
分支;就让张三提个Merge
申请,让他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
,进行新一轮的dev
很开心;可以删除这个本地zhang_san
分支和远程zhang_san
分支,保持分支干净;也可以继续在zhang_san
分支做新的开发,提交新的Merge_Request
,进行新一轮的Code Review
。code review 就是代码审查,你可以百度搜索一下,有很多文章
你写的代码,别人看,别人的代码,你也看
review 过了,再上线
代码审查,一般大公司都需要code review。如果代码有问题或者不符合规范,修改正确后再上线。
要你命三千:老代码中的那些坑