这几天,递交了keystone, nova, quantum, melange 的几个patch:
https://review.openstack.org/#dashboard,2634
一些体会:
1、openstack 采用git 管理代码,所有代码,都是必须通过 review,然后再递交,使用工具 Gerrit
我们团队之前采用svn,后来部分项目转向了git。但是代码评审这一块做得不好,没有强制review,导致代码质量无法控制。
openstack采用 Gerrit 工具和流程,必须经过 review, verified, approved,然后merge,这个准备在团队中,使用一两个项目开始做,然后推广。
2、强调单元测试,必须进行,尽量尽量单元测试覆盖,降低了出错的几率,提高了开发效率
做单元测试,对于开发人员来说,还是有一些抵触,很多人认为增加了工作量。但是从 total time 来说,肯定是减少了开发和测试时间,毕竟代码不是一次性开发出来就行的。
3、强调 PEP8 和 pylint,并且做了自动化测试,空行、空格等,都严格校验