phabricator评审流程(开发人员项目经理篇).docxVIP

苏州市科远软件技术开发有限公司上海分公司 Phabricator评审流程 版本0.1 徐玉龙 2015/1/14 版 本 号： V0.1 文 档 等 级： 参考手册 编 制： 审 核： 说明 A．角色 代码修改人：liming 代码审核人：lihuaifang 其他开发人：qiaozhi phabricator管理员:wuxiaona B．配置 对于开发人员，需要进行phabricator客户端配置后才能触发提交前评审流程，详见《Phabricator客户端部署（开发人员篇）.docx》。其他人员如项目经理，直接在Phabricator上登陆即可使用。 C．使用phabricator前须注册有效账户，向对应的phabricator管理员申请权限，输入密码激活个人账户，会收到如下邮件，输入密码即可使用。 D．本文档演示了最常见的phabricator提交前评审和提交后评审流程，开发人员熟悉了之后可灵活变化。 提交前评审（Differential） 步骤1：创建代码评审 Liming修改了代码，将修改的文件提交评审: 1)原始arc工具提交评审 修改内容见图1.1的svn diff命令结果，代码修改之后通过arc diff命令提交评审。 arc diff命令会弹出文本编辑器，根据实际情况填写相关信息，重点是审核人（Reviewers必填字段，两个以上审核人用逗号分隔），CC抄送相关人。 图1.1 编辑器内填好后保存退出（图1.2），则Phabricator自动创建审核任务并发送邮件给相关人 图1.2 2) karc工具提交评审 使用karc diff创建评审,选择是否使用上次评审内容作为模板,第一次操作不会出现此界面 图1.3 我们选择n,弹出nodepad++界面,内容如下: 图1.4 图1.5 按照svn提交规范,填写模板,保存退出,如果项目中存在很多零时文件,会出现下面选项,一般选择n 图1.7 后续同上. 3)图形界面创建评审 鼠标选中要提交评审的文件夹,右键选择arcanist create 图1.8 图1.9 图1.10 步骤2：评审人进行代码评审 Lihuaifang收到评审请求邮件： 抄送人Qiaozhi收到类似邮件（抄送人没有审核权，仅仅知会到而已）: Lihuaifang收到邮件后，登陆Phabricator，看到有一个新增评审任务（即提交前评审Differential），如图1.3。 图1.3 点开任务D8 test example明细如图1.4，lihuaifang觉得代码修改得有问题，给予驳回。 图1.4 步骤3:开发人员更新代码评审 liming收到驳回邮件： 之后，Liming根据评审意见（邮件和phabricator都可以看到lihuaifang具体审核意见），重新修改代码，并继续用arc diff命令添加再次修改的注释，保存退出后，李怀芳收到再次修改的评审请求邮件： 1)arc工具更新评审 Notepad++填写好更新说明,保存退出,即可 2)karc diff工具更新评审,出现下面选项选择N即可 3)图形界面更新评审 选择要提交评审文件夹,右键选择arcanist update 步骤4：评审人员再次评审代码 lihuaifang登陆phabricator再次评审….. 图1.5 如此直到评审通过，lihuaifang接受了该条修改，评审过程通过。 步骤5：开发人员提交代码 Liming收到评审通过邮件后，可以用arc commit提交代码，phabricator自动关闭评审。 1)arc工具提交代码 arc commit –revision D8 2)karc工具提交代码 karc diff –update D8 3)图形界面提交代码 在svn工程下选择任意文件或者文件夹右键,选择arcanist commit 图1.6 回顾整个流程如下： 图1.7 提交后评审（Audit） 相对提交前评审，提交后评审流程上更简单宽松，唯一的约束是代码评审人接受了该svn commit才能关闭该条评审。 步骤1：开发人员提交代码并创建代码评审 Liming修改了代码并svn commit，按照格式填写了Auditors: name1,name2(图2.1)，此时会自动触发Phabricator创建一条audit（即提交后评审任务） 图2.1 注：也可命令行E:\test4jenkins\code svn ci -m Auditors:lihuaifang，效果相同。 步骤2：评审人员进行代码评审 Lihuaifang收到邮件并登陆Phabricator查看修改。 图2.2 Lihuaifang认为修改有问题，予以驳回（填上具体的驳回原因），要求liming重新修改。 图2.3 步骤3： Li