English | 简体中文
我们坚信code review所带来的价值：code review不仅可以提高质量、代码的可读性，同时也能不断提高开发者的水平， 促进开发者的编码能力，做出更好的设计。
以下是蓝鲸团队review issues和PRs/MRs的相关指引。
最重要的一条：欢迎在任何时候，反馈问题、反馈需求
提交前相关规范请审阅commit-spec
Code review虽然会让特性与问题解决稍稍延后，同时也会造成额外审阅工作。因此蓝鲸团队希望积极参与项目相关人员也是 积极的审阅者，同时也期望相关的审阅者也具备相关领域的专业知识，以期提高处理效率。
当PR/MR被提交之后，审查人员需要对该PR/MR进行快速分类，例如关闭重复，识别是否存在简单的用户错误，打上标签， 确认该PR/MR由哪些更具专业知识的审查人员进行review。
如果PR/MR被拒绝了，审查人员需要向发起者提供足够的信息反馈，解释为什么会被关闭。
在review的过程中，PR/MR的发起者请积极回答审查人员的疑问，进行评论。如有需要，对提交内容进行合理的调整。
审查人员在工作日内对于PR/MR处理要及时。在非工作日内，大家必须意识到，相关的处理会有所延时。
所有的参与人员不可能永远在处理相关的需求，如果审查人员因为时间关系没能立刻处理，建议在PR/MR的讨论中反馈处理时间。 对于PR/MR的作者，合理的做法是和审查人员积极磋商合理的deadline，或者是否有其他专业的审查人员可以处理，协商PR/MR 的交接。
PR/MR达成以下标准时会被Merge:
- 审查人员没有提出反对或者整改意见
- 所有反对的建议或者调整的建议已经合理处理
- 至少有一个分支的维护者赞成合入
- 具备相关的文档和测试