NeutronReviews
代码审查是所有 OpenStack 项目的关键组成部分。Neutron 接受来自众多不同背景、雇主和经验水平的人员提交补丁。代码审查提供了一种强制项目一致性的方法,并允许谨慎地引入新贡献者的贡献。
Neutron 代码审查实践
Neutron 遵循所有 OpenStack 项目制定的 [代码审查指南]。预计所有审查员都将遵循该页面中制定的指南。
Neutron 规范审查实践
除了代码审查之外,Neutron 还维护一个 BP 规范 git 仓库。关于如何使用此仓库的详细说明请参见 此处。预计 Neutron 核心团队成员将积极审查推送到规范仓库进行审查的规范。下面提供了一些关于此过程的指南
- 一旦规范被推送,预计在第一位核心审查员审查后至少 3 天才能批准该规范。这允许其他核心成员审查该规范。
- 对于核心团队认为具有高或关键重要性的 BP,核心审查员可能会根据他们的主题专业知识进行分配。
- 规范优先级将由 PTL 在规范获得批准后与核心团队一起设定。
指定审查员
任何人都可以使用 gerrit 指定代码审查员。此外,任何人都可以审查 gerrit 中的任何补丁。但是,对于优先级为关键或高的项目,Neutron 核心团队将指定 2 名核心审查员来审查这些项目。这些项目通常被认为是社区中至关重要的,因此确保它们具有与开发覆盖率一样多的审查覆盖率对项目来说是有意义的。核心团队自行决定将核心审查员分配给中等或更低优先级的项目。
跟踪审查统计信息
Stackalytics 提供了一些不错的界面来跟踪审查统计信息。下面提供了链接。这些统计信息不仅用于跟踪核心审查员的统计信息,还用于跟踪潜在未来核心成员的审查统计信息。