Code Review的意义

code review是现代软件开发非常有意义的一种技术管理方法,其价值不止在于代码的准入和找bug,也是所有人技术交流和成长的一种手段,总结CR的意义,不限于:

  • 找出可能的坑
  • 帮助新人成长
  • 互相督促代码质量
  • 互相学习设计理念及编码思路

理想情况下所有代码合并都必须进行code review,并且必须CC整个组,让你的每一个合并都更认真地对待

phabricator

诞生于Facebook内部,是一套基于Web的软件开发协作工具,包括代码审查工具Differential,资源库浏览器Diffusion,变更监测工具Herald,Bug跟踪工具Maniphest和维基工具Phriction。Phabricator可与Git、Mercurial和Subversion集成使用。

其本身是PHP开源程序:https://github.com/phacility/phabricator

系统配置

PHP环境请自行google,这里记录nginx配置

server {

set $content "~/dev-lib/phabricator/webroot";

listen 80;

server_name phabricator.tangkunyin.com;

root $content;

location / {
index index.php index.html index.htm;
rewrite ^/(.*)$ /index.php?__path__=/$1 last;
}

location ~ \.php$ {
fastcgi_pass 127.0.0.1:9000;
fastcgi_index index.php;
fastcgi_intercept_errors on;
include /usr/local/etc/nginx/fastcgi.conf;
}

}

运行后,会提示通过命令设置Mysql信息,一般情况下直接设置密码即可

接着,会让你更新数据库信息,同样按提示操作即可


如此phabricator就算安装完成了,接下来根据引导配置一下系统各项参数。以下这些看情况配置就好

项目配置

先下载并配置环境变量:https://github.com/phacility/arcanist
再设置帐号信任:

arc set-config default http://phabricator.tangkunyin.com/
arc install-certificate http://phabricator.tangkunyin.com/

执行后会提示到网站上找到对应的token,找到后粘贴输入。注意提示的网址应该是:http://phabricator.tangkunyin.com/conduit/login/,如果不是重新登录以加载新的配置

最后在项目中创建.arcconfig文件并配置基础项,完整配置请自提自:arc get-config --verbose

{
"phabricator.uri" : "https://phabricator.d.xiaomi.net/",
"history.immutable": false
}
review步骤:

1、git commit你的修改
2、开始review

arc diff commit_id --create

或修改

arc diff commit_id --update D12306 (这儿的D12306是上一次review时的revisionId)

3、填写review信息

4、现在你会收到邮件,也可以在网站上看到你的review,请大家评审吧

5、merge仍然要在gitlab中完成,merge消息中要带上phabricator的review地址,证明这次merge是review过的,收到了accept的review才进行合并

需要注意的是我这里并没有提到邮件发送配置,实际过程是要配置的,不然上边的邮件肯定发不出去

具体参考这篇文章:https://cloud.tencent.com/developer/article/1609447

平台操作资料:Phabricator Code Review操作手册