review

Code review的原则

这些原则参考了清风(前豆瓣工程师)在infoQ上的演讲,另外加上了个人的思考。

  1. 言行文明,就事论事,对事不对人,毕竟每人都写过烂代码。
  2. commit msg尽量详尽,让reviewer对patch的意图有整体的认识。
  3. 尽量提较小的、功能内聚的patch,既方便revert,又不会引起reviewer的恐慌。
  4. 如果reviewee接受reviewer的意见,最好的意见下方注明“done”,表示对应的修改已经完成,方便reviewer对新patch再次review。
查看详细

review.openstack.org被墙后怎么办?

大家发现review.openstack.org:29418 在神奇的国度突然不能访问了,怎么办?

这里列一些思路:

  1. 如果有国外的服务器,那么挂VPN,开ssh隧道都是可以的。不过ssh隧道现在似乎也被封杀了,起码我的不行。
  2. 如果没有国外服务器,怎么办?
    不要用默认的ssh://协议,使用https://,很简单:

这么搞:

1 . https://review.openstack.org/#/settings/http-password 设置一下http的密码,可以生成随机密码。

2 .

git clone https://{username}@review.openstack.org/openstack/nova
查看详细