代码风格
每种语言都会有自己特定的风格,每个人也会会有自己的风格,那么在协作的项目中,我们就需要去遵循大家都认可的规范。
熟悉语法和风格
在提交代码到代码审查系统前,请在本地开发环境使用Puppet-lint
工具检查一遍。
如何标记弃用(Deprecation)
所有patch必须保持向后兼容(backward compatible)
这意味着:
- 不能破坏原有接口(参数弃用至少保持一个周期,并要添加warning信息)
- 不能改变参数的默认值(除非有一个好理由,并在commit消息里解释清楚原因)
代码一致性
目前社区维护了大量的modules,我们需要保持代码的一致性,这体现在使用统一风格的代码和注释,每个独立参数选项保持唯一。例如,在添加新参数前,请查找是否在其他类或者模块中已经实现,如果没有,则保持和其他参数相同风格的方式添加。
空值的参数
当你需要设置一个空的(nil)参数时,使用undef
值。不要使用’’或者false。
文档
- 确保所有的参数都添加了正确的文档,lint会检查这些文档是否被添加。
- 尽可能地保持examples.pp文件更新
- 必须情况下,为你的代码添加注释(temporary workarounds, TODO等)