gerrit

centos 7 部署 gerrit + LDAP + gitweb + MySQL

前言

核心需求是部署一个 Gerrit 做 code review。操作系统是 Centos 7,由于公司目前没有 LDAP 服务,所以只能自建。另外还需要 gitweb 方便在页面上查看代码。数据库使用最熟悉的 MySQL。为什么要强调 centos 7,因为版本不同,安装过程差别就比较大,centos6 和 7 上 ldap 安装就不大一样,和 debian 那更不一样,所以特别点出,以防用错。

全部部署完成后的版本号:

mariadb-server-5.5.47-1.el7_2.x86_64  # mysql
openldap-2.4.40-9.el7_2.x86_64
查看详细

Code review的原则

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

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