用GitLab的Merge Request作代码评审

在目前已使用的质量内建的工程实践中不能否认的一个实践为代码审查 它被用做提升产品交付质量和提升开发过程效率的有效措施。 Git又是目前当红的源码管理工具，若你的团队目前已经选用了GitLab来做为托管工具，那此文中你能够学到如何经过GitLab的Merge Request（合并请求）进行代码审查以及咱们遵循的现有代码审查最佳实践来改进工做流程。

在咱们讨论如何进行代码审查以前，让咱们先来回顾一下代码评审的通常原则。

代码评审的通常原则

• 代码评审是任何开发过程当中不可或缺的一部分 - 将其打印出来并放在墙上以便记住。可参考以前写过的你的代码评审须要来一次清单革命！
• 代码评审是在小段的逻辑完整的代码片断上执行的，例如功能，任务，错误修复，改进等。
• 只有经过审核的代码才会发送到测试部门。
• 该项目的全部开发人员都会进行代码评审，不管他们的级别如何。
• 项目的全部开发人员都应该经过代码评审，不管他们的级别如何（初级开发人员也应该审查经验丰富的中高级专家的代码）。

接下来咱们将介绍如何使用GitLab提供的工具来进行代码评审。
GitLab中的merge request指的是把代码从一个分支合并到另外一个分支上作的操做。

建立一个Merge request会涉及到的主要参数为：

• source branch
• target branch
• title
• description

• assignee

  
  
  content_Creating-merge-request.png

使用Merge Request时的操做步骤：

1. 编写代码并将其推送到单独的分支。
2. 为主要开发分支建立合并请求。 Assignee以及说明字段和评论中被提到的那些人将经过电子邮件通知合并请求。若是须要某一位开发人员关注，你能够在描述字段中@该名开发人员。
3. 等到MR被接受或拒绝，并提供有关必要修复的评论。
4. 参与有关修复的讨论。 （GitLab容许回复评论）
5. 修复。
6. 将更改推送到你的分支。
7. 打开一个新合并若是最后一个MR被关闭（若是合并请求未关闭，它将自动更新，直到最后一次提交为止）。
8. 经过注释合并请求或以其余方式报告已实施的修复。

content_reviewing-merge-requests.png

应该将Merge Request分配给谁

对于合并请求，它们的分配取决于各类因素。根据项目的人数和专业水平，能够有不一样的选择。所以，若是您是团队中惟一的开发人员，请为本身分配合并请求。

不然，请与另外一位在项目中独立的开发人员交谈，并让他审查彼此的代码。文档审查一般也是必要的，由于在您执行此操做后，您将确保其余开发人员能够在必要时使用您的代码。

若是您是项目的两名开发人员，请相互分配合并请求。若是有三个或更多开发人员，您能够自由选择。

你的团队能够在工做日的开始和结束时或根据要求随时进行代码审查。团队能够决定什么时候进行代码审查，最重要的是团队成员之间的持续协做。

用Merge Requests产生的代码评审如何进行更精细化的流程管理以后能够继续讨论。