openstack 开发和递交代码注意事项

这几天,递交了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,并且做了自动化测试,空行、空格等,都严格校验

此条目发表在OpenStack分类目录,贴了标签。将固定链接加入收藏夹。