使用 Phabricator 来 code review

Phabricator 是最早发源于 Facebook 的代码 review 工具，后来创始人离开 Facebook 独立开发这款工具。Phabricator 和同类型的 Google 的 gerrit 有着相似的功能。

Phabricator 会将所有文件 diff 直接展开，而 Gerrit 只有一个文件列表，需要点每一个文件看具体修改；
admin 的权限设置区别
Phabricator 可以附加 unit test 和以及格式检查和规范的工具 (e.g. JSLint), 然后用户在上传 diff 的时候，Phabricator 会自动用新版本代码去跑 unit tests, 以及运行 JSLint, 然后在 code review 里面就可以看到 unit tests 有几个 failure 和 error, 以及 JSLint 的报警信息

Arcanist

Phabricator 提供了个命令行叫 arc （Arcanist），使用 arc 命令来连接开发者和 Phabricator 服务器。

安装 Arcanist

Mac OS

brew install node
npm install -g arcanist
arc help
arc upgrade

Ubuntu/Debian/Linux Mint

在 Linux 下使用

sudo apt update && sudo apt-get install php7.0-cli php7.0-curl php-pear

# 任意其它目录也可以
mkdir ~/phabricator
cd ~/phabricator
# 如果下载慢的话，可以先尝试设置一下 proxy: export https_proxy="ip:3128"
git clone https://github.com/facebook/libphutil.git
git clone https://github.com/facebook/arcanist.git

# 编辑~/.bashrc，加入如下一行，之后 source ~/.bashrc
export PATH="$PATH:$HOME/phabricator/arcanist/bin/"
# 如果没有项目配置，可以运行下面的命令设置全局的参数：
arc set-config default https://<phabricator server address>
arc install-certificate
# 按照屏幕提示，访问 http://<phabricator server address>/conduit/token/ , 把 token copy/paste 下来
# 在~/.bashrc 里添加下面两行
export EDITOR=vim
alias arc='LC_ALL=C arc'
# 并在命令行执行
source ~/.bashrc

Windows

自行处理：https://www.jianshu.com/p/a6ee738da1aa

配置

一般情况下在 HOME 目录下会产生一个 .arcrc 配置文件，格式类似下方：

{
  "config": {
    "default": "https://<phabricator server address>",
    "editor": "vim"
  },
  "hosts": {
    "https://<phabricator server>/api/": {
      "token": "cli-rdm2xxxxxxxxxxxx"
    }
  }
}

可以使用如下命令来更改该配置

arc set-config phabricator.uri "<phabricator server>"
arc set-config editor "vim"

本地开发流程

如下：

# 本地开发，新建 feature
arc feature feature_1
# 或 git checkout -b feature_1
# 修改本地文件，然后提交
git add
git commit

# 提交 code review
arc diff
# 通过 --reviewers name 参数可以指定 reviewer

# 这里可以和一个 Task（比如 T123）关联起来，在 Summary 那个域里填上 Ref T123，或者 Fixes T123
# 两者的区别是 Fixes 在代码提交后会自动关闭 Task，Ref 不会
# 也可以提交 code review 之后，在浏览器里操作

# 本地修改后再次提交 code review（这个 revision 是上次创建 code review 时的 id，比如 D1252）
arc diff --update 1252
# --update 1252 也可以不加，默认会合到上一个 review 中

# reviewer 已经 Accept 之后，提交代码（不要用 git push！）：
arc land feature_1
# 如果报错找不到对应的 revision，带上 revision 号（比如 1252）运行：
arc land --revision 1252
# land 之后，feature branch 会被自动删除
# land 使用后会将当前分支 merge 或者 squash merge 到 master 分支，提供详细的提交信息，推送到 master 并且删除本地分支

Q & A

问题： No changes found. (Did you specify the wrong commit range?)

答： Arcanist(arc) 是一个 pre-commit 或者说 pre-push 代码审查工具，一旦本地分支被 push 到 origin，这就打破了 Arcanist 工具的意义，但是可以用以下方法来补救：

删除当前分支的远端分支 git push origin --delete branch_name
在使用 arc diff 时告诉分支需要和哪一个分支对比 arc diff origin/master
或者显示指定 land 的目的地 arc land --onto master