使用arc进行code review
https://secure.phabricator.com/book/phabricator/article/arcanist_quick_start/
使用流程:
流程
本部分来自arcanist
git checkout -b feature_xxx— 创建一个新分支,并修改代码git add . && git commit -m tt— 将该提交的都提交上去arc diff [last_commit]- 此命令会将 last_commit 到最新的 commit 之间的改动的所有代码发送到 phabricator 平台,并创建一个 revision;
 - 如果没有指定 last_commit,则会使用 master 分支来作为 last_commit 的起点。
 - 运行命令后会弹出一个编辑面板,在此面板中你需要指定 reviewers,即指定一个或多个给你 review code 的人
 - 如果指定了多个 reviewers,则其中任何一个 review 通过就可以,不用全部 review 通过
 - 另外编辑面板中也有一个 test plan 选项,国外人一般每个项目都有测试,所以此项是必填的,如果没有测试,我们只需要随便设置一个字段即可,比如设置成 
no或skip - 如果觉得指派reviewer的用法不太方便,可以执行
arc diff --preview根据生成的diff页面进入配置,生成revision. 
- 以上操作之后,系统会发送邮件通知 reviewers 去 review 你的代码,你也可以通过 
arc list来查看当前 review 的状态 - 后续
- 如果 review 没有通过,你需要在原来的基础上修改,修改完并 commit 之后需要执行 
arc diff [last_commit] --update D(id)继续 review - 如果 review 通过了,只需要运行 
arc land --onto some_branch, arc 会将你当前分支合并到 master 上,并删除当前分支 
 - 如果 review 没有通过,你需要在原来的基础上修改,修改完并 commit 之后需要执行 
 
使用arc进行code review的更多相关文章
- 有人实践过 Phabricator 以及 Arcanist 作为 code review 的工具么?(转)
		
作者:覃超链接:http://www.zhihu.com/question/19977889/answer/13539702来源:知乎 平时就经常实践. 整个公司的code review就是使用这个. ...
 - 我们是怎么做Code Review的
		
前几天看了<Code Review 程序员的寄望与哀伤>,想到我们团队开展Code Review也有2年了,结果还算比较满意,有些经验应该可以和大家一起分享.探讨.我们为什么要推行Code ...
 - Code Review 程序员的寄望与哀伤
		
一个程序员,他写完了代码,在测试环境通过了测试,然后他把它发布到了线上生产环境,但很快就发现在生产环境上出了问题,有潜在的 bug. 事后分析,是生产环境的一些微妙差异,使得这种 bug 场景在线下测 ...
 - Git和Code Review流程
		
Code Review流程1.根据开发任务,建立git分支, 分支名称模式为feature/任务名,比如关于API相关的一项任务,建立分支feature/api.git checkout -b fea ...
 - 如何搭建开源code review gerrit服务器
		
搭建环境:Ubuntu 14.04 一.环境准备 1.Java环境 gerrit依赖,用于安装gerrit环境. 下载:jdk-7u79-linux-x64.tar.gz http://www.ora ...
 - Code Review Tools
		
Code Review中文应该译作“代码审查”或是“代码评审”,这是一个流程,当开发人员写好代码后,需要让别人来review一下他的代码,这是一种有效发现BUG的方法.由此,我们可以审查代码的风格.逻 ...
 - code review作业
		
下面是对结对编程队友12061166 宋天舒的code review 五个优点: 1.代码的风格优秀,注释不多,但是必要的注释还是有的,比如: // 三种模式 // mode1仅统计单个单词 // m ...
 - 15个最佳的代码评审(Code Review)工具
		
代码评审可以被看作是计算机源代码的测试,它的目的是查找和修复引入到开发阶段的应用程序的错误,提高软件的整体素质和开发者的技能.代码审查程序以各种形式,如结对编程,代码抽查等.在这个列表中,我们编制了1 ...
 - Code Review 五问五答
		
Code Review 是什么? Code Review即代码审查,程序猿相互审核对方的代码. Code Review能获得什么好处? 提高代码可维护性 你写的代码不再只有编译器看了,你得写出审核人能 ...
 
随机推荐
- homebrew 无法安装,提示不能在根目录下使用
			
sudo chown -R $(whoami) /usr/local 把/use/local的owner換成自己,就有write權限了 whoami就是一個命令,會echo當前登錄用戶的名字.當然你知 ...
 - [android] AndroidManifest.xml - 【 manifest -> permission】
			
在 API Level 1 时被引入 语法: <permission android:description="string resource" android:icon= ...
 - C++ 运算符重载二(一元运算符重载)
			
//一元运算符重载 #include<iostream> using namespace std; class Point { public: Point(int x,int y){ th ...
 - ubuntu 按键替换 Control_R to Left
			
ubuntu 按键替换 Control_R to Left 1 查看当前键盘布局 $xmodmap -pke keycode 105 = Control_R NoSymbol Control_Rkey ...
 - ACM计算几何模板——圆和球
			
#include <iostream> #include <cmath> using namespace std; #define eps 1e-10 /********** ...
 - LoadRunner压力测试心得总结
			
一.虚拟用户迭代一次的时间对整个压力场景的影响. 1.虚拟用户迭代一次的时间大于等于压力场景的上行周期. 此种情况,在压力场景的上行周期中,所有虚拟用户根据压力场景设置的策略全部依次运行.压力场景的上 ...
 - MathType怎么编辑双箭头
			
很多的数学相关工作者在写文章或论文的时候常常会用到数学公式编辑器.MathType就是一款深受大家欢迎的公式编辑器.很多的用户在使用过程中会用到双箭头符号来表示推理过程,但是怎么编辑又不知道,下面本教 ...
 - Visual Studio Code调试node.js:无法在PATH上找到运行时的node
			
首先,环境变量Path中加入nodejs的路径: 验证nodejs是否已经加入环境变量: 接着,重新启动Visual Studio Code, 试一下,是不是好了~ 附录:Visual Studi ...
 - ReactNative iOS源码解析
			
http://awhisper.github.io/2016/06/24/ReactNative%E6%B5%81%E7%A8%8B%E6%BA%90%E7%A0%81%E5%88%86%E6%9E% ...
 - android classloader双亲托付模式
			
概述 ClassLoader的双亲托付模式:classloader 按级别分为三个级别:最上级 : bootstrap classLoader(根类载入器) : 中间级:extension class ...