Sahara/ReviewChecklist

此页面包含审查者在审查 Sahara 项目补丁时应牢记事项清单，它基于 OpenStack 的 代码审查清单。

通用代码审查清单

1. 代码应符合 HACKING 中的所有要求。
2. 代码应“Pythonic”，并与周围代码风格一致，以使代码更统一且更易于阅读。
3. 添加新文件时
   1. 如果文件未通过 setup.py 安装，但应包含在 tarball 中，请务必将其添加到 `MANIFEST.in` 中。
4. 提交消息和变更拆分
   1. 遵循 Sahara/GitCommits 的建议。
   2. 如果补丁修复了错误，它应引用错误报告。
   3. 如果补丁实现了新功能，它应引用蓝图。在合并补丁之前，蓝图应获得批准。
5. 批准时机
   1. 大多数补丁在批准前需要两个 +2，请不要急于让其他贡献者能够查看补丁。
   2. 如果补丁已经获得批准，但需要一个微不足道/简单的 rebase 才能合并，则无需等待第二个 +2，因为该补丁已经获得了两个 +2。

非核心开发者须知

• 在您审查时，您可能会注意到一个审查已经获得了其他审查者多个 +1，通过了功能测试等，但代码仍然未被合并。由于只有核心开发者才能批准代码进行合并，您可以帮助加快流程，通过引起核心开发者的注意，并让他们知道有一个审查获得了大量积极的反馈，需要最终批准。