Phabricator评审流程开发人员项目经理篇.docx

Phabricator评审流程开发人员项目经理篇.docx

《Phabricator评审流程开发人员项目经理篇.docx》由会员分享，可在线阅读，更多相关《Phabricator评审流程开发人员项目经理篇.docx（13页珍藏版）》请在冰豆网上搜索。

Phabricator评审流程开发人员项目经理篇

苏州市科远软件技术开发有限公司上海分公司

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的svndiff命令结果，代码修改之后通过arcdiff命令提交评审。

arcdiff命令会弹出文本编辑器，根据实际情况填写相关信息，重点是审核人（Reviewers必填字段，两个以上审核人用逗号分隔），CC抄送相关人。

图1.1

编辑器内填好后保存退出（图1.2），则Phabricator自动创建审核任务并发送邮件给相关人

图1.2

2）karc工具提交评审

使用karcdiff创建评审,选择是否使用上次评审内容作为模板,第一次操作不会出现此界面

图1.3

我们选择n,弹出nodepad++界面,内容如下:

图1.4

图1.5

按照svn提交规范,填写模板,保存退出,如果项目中存在很多零时文件,会出现下面选项,一般选择n

图1.7

后续同上.

3）图形界面创建评审

鼠标选中要提交评审的文件夹,右键选择arcanistcreate

图1.8

图1.9

图1.10

步骤2：

评审人进行代码评审

Lihuaifang收到评审请求邮件：

抄送人Qiaozhi收到类似邮件（抄送人没有审核权，仅仅知会到而已）:

Lihuaifang收到邮件后，登陆Phabricator，看到有一个新增评审任务（即提交前评审Differential），如图1.3。

图1.3

点开任务D8testexample明细如图1.4，lihuaifang觉得代码修改得有问题，给予驳回。

图1.4

步骤3:

开发人员更新代码评审

liming收到驳回邮件：

之后，Liming根据评审意见（邮件和phabricator都可以看到lihuaifang具体审核意见），重新修改代码，并继续用arcdiff命令添加再次修改的注释，保存退出后，李怀芳收到再次修改的评审请求邮件：

1）arc工具更新评审

Notepad++填写好更新说明,保存退出,即可

2）karcdiff工具更新评审,出现下面选项选择N即可

3）图形界面更新评审

选择要提交评审文件夹,右键选择arcanistupdate

步骤4：

评审人员再次评审代码

lihuaifang登陆phabricator再次评审…..

图1.5

如此直到评审通过，lihuaifang接受了该条修改，评审过程通过。

步骤5：

开发人员提交代码

Liming收到评审通过邮件后，可以用arccommit提交代码，phabricator自动关闭评审。

1）arc工具提交代码

arccommit–revisionD8

2）karc工具提交代码

karcdiff–updateD8

3）图形界面提交代码

在svn工程下选择任意文件或者文件夹右键,选择arcanistcommit

图1.6

回顾整个流程如下：

图1.7

提交后评审（Audit）

相对提交前评审，提交后评审流程上更简单宽松，唯一的约束是代码评审人接受了该svncommit才能关闭该条评审。

步骤1：

开发人员提交代码并创建代码评审

Liming修改了代码并svncommit，按照格式填写了Auditors:

name1,name2（图2.1），此时会自动触发Phabricator创建一条audit（即提交后评审任务）

图2.1

注：

也可命令行E:

\test4jenkins\code>svnci-m"Auditors:

lihuaifang"，效果相同。

步骤2：

评审人员进行代码评审

Lihuaifang收到邮件并登陆Phabricator查看修改。

图2.2

Lihuaifang认为修改有问题，予以驳回（填上具体的驳回原因），要求liming重新修改。

图2.3

步骤3：

Liming收到邮件，并登陆Phabricator查看lihuaifang的驳回意见（图2.4）。

之后，liming重新与相关人员讨论并修改代码，或者通过跟lihuaifang交流（线上：

通过phabricator；线下：

面对面交流），最终达成共识不需要重新修改。

图2.4

步骤4：

lihuaifangAccept，该条评审结束。

图2.5