code review基础方案

1.pre-commit-review

采用feature分支及merge-request的方式进行code review

使用流程：

1》开发人员创建自己的feature分支进行开发，所有code提交到自己的feature分支上；

2》当一个feature开发完成时，将自己的feature分支push到git远程代码库的同名feature分支上；

3》提交一个merge-request申请

step1> 在gitlab上项目的merge requests页面点击按钮，

step2> 在出现的对话框中选则source和target branch信息，如下

step3> 点击按钮

step4> 填写title，description，assgin to信息，其中在description信息中，可以使用@的形式添加需要做code review的人员，assgin to的人员则完成最后的merge操作

step5> 点击

4》 merge request创建完成后，被@到的人员及assgin to的人员将会收到邮件通知

5》点击邮件中的“view it on gitlab”即可进入gitlab的merge request页面，查看代码变更信

6》鼠标指到某一行，在左边会出现一个信息添加标志，点击即可出现comments提交框

7》提交comments之后，其他所有参与code review的人员都将收到邮件通知

8》提交merge request的人员可根据comments信息，提交新的commit到feture分支上，merge request会自动更新feature分支到最新版本

9》 code review完成后可由assgin to指定的人员执行merge操作

2. post-commit-review

方案一：逐个commit节点进行code review

1》开发人员将自己本地的commit推送到服务端develop分支上，gitlab发送邮件给git库中所有组员

2》组员可在gitlab中commits页面，进入commit详情页面，查看code change,进行review

方案二：为需要做code review的commit节点单独创建分支，提交merge request进行code review，此处仅做code review，不执行merge操作；

1》为需要做对比的两个commit节点分别创建分支，如下图所示，如需做commit1~commit2之间的code review，可如图分别为两个节点创建branch-1，branch-2

2》创建merge request，source选择 branch-2, target选择branch-1创建一个merge request，方法如pre-commit-review中所示；

3》相关人员可在这个merge request上做code review的comments

4》开发如需根据comments修改代码，可在branch-2上修改，重新提交后，merge request也随之更新branch-2，所有code review结束后，创建人可将merge request关闭，将branch-2上的code merge到主开发分支即可；