[关闭]
@chenpbh 2018-05-03T04:01:18.000000Z 字数 1693 阅读 478

Arcanist CodeReview 使用帮助说明 1.0

公共文档


变更历史

版本 日期 人员 变更内容
1.0 2018-05-02 陈鹏 版本初定义

1、Arcanist配置

详细配置说明,可以直接参见官方文档(翻译中文)Arcanist 快速开始

1.1 windows

官方也有针对windows配置的说明,具体见Arcanist 用户指南: Windows,在此教程,提供了快速的配置,主要针对php配置作了相应的处理。

1.1.1 配置hosts文件

由于当前code review处于试运行阶段,先采用直接修改hosts方法支持域名访问。

image_1ccftvtub1hlj1i4r4divddkpkhq.png-33.3kB

image_1ccftrks81ah15fngg6aumr0ugd.png-64.5kB

  1. 192.168.2.213 review.bitnei.cn

1.1.2 下载相关组件

为了快速配置,我们固定将组件存入到E:\review目录(默认的windows命令行可能不支持mv操作,如果不行,直接通过拷贝实现)

  1. #
  2. cd e:
  3. #下载组件,
  4. git clone http://review.bitnei.cn:8080/source/phabriatorConfig.git
  5. # 重命名目录
  6. mv phabriatorConfig e:\review

1.1.3 配置环境变量

将php和arcanist的可执行目录增加到PATH,即:

image_1ccfsepk81k5a1cepg8k17s6gbr92.png-48.3kB

1.1.4 测试

image_1ccfsgs9r1vff1fs31a4k7281ahaf.png-25kB

2 系统简单使用说明

2.1 登录系统

通过浏览器,打开review.bitnei.cn:8080,输入用户名和密码

image_1ccftair03t29kps0bcaool3cc.png-36.9kB
image_1ccftbrus1c8irej4r11tv49tqcp.png-165.2kB

2.2 修改语言环境

image_1ccfte2fehop13ehaatmnqvmcdm.png-67.2kB

2.3 修改VCS密码

vcs密码即为svn、git的密码,这里不能和登录密码相同。

image_1ccfthibsrje1sob1lc8udq1aooej.png-113.2kB

3 项目配置

要求在项目根目录,创建一个.arcconfig文件(此工作应由工程负责人处理,然后提交到git仓库上),配置内容如下

  1. {
  2. "phabricator.uri" : "http://review.bitnei.cn:8080/"
  3. }

4 审核流程

4.1 提交审核

image_1ccfna7l41b7j7kh1tv217fhjhe9.png-5.5kB

image_1ccfnckog7ll1k6hble1n1m1djpm.png-13.9kB

a. 若当前项目存在未增加到本地缓存的文件,会出现这个提示,在这里,我们输入y即可

image_1ccfnif8u1dr1bnu19ln1n65bnh13.png-22.7kB

b. 然后arc会通过调用外部的编辑器(linux和mac默认会调用vi),对提交的审核信息进行编辑,编辑完之后保存,然后关闭编辑器,arc就会自动将审核内容提交

image_1ccfnof1tq0fdmpuv1cn915pd1g.png-52.8kB

c. 提交成功

image_1ccfo00hs1msb1kb01egk19hii5d1t.png-36kB

d. 查看提交的审查列表

image_1ccfobmot1hlg1id71j6o14ln14ra2a.png-6kB

4.2 审核人员审核

image_1ccfog7nd1dd51d0s113fspv1os02n.png-75.1kB

4.3 提交代码至仓库

image_1ccfpi3i4e161p14danekk1ci468.png-7.7kB

image_1ccfpnfo8g4boa70e1omp7b385.png-85.1kB

A 附录

A.1 常见问题

A.1.1 为什么使用arc land之后,不需要再操作git push xxx xxx?

答: arc land已经集成了git push操作

A.1.2 审查不通过之后,开发人员应该如何操作?

答: 开发人员需要根据审查人员反馈的情况,对代码处理之后,重新通过arc diff提交审查即可。

A.1.3 审查通过之后,向仓库推送代码里发送存在冲突,开发人员应该怎么处理

答: 合并之后,要通过arc diff再次发起审核

添加新批注
在作者公开此批注前,只有你和作者可见。
回复批注