Code Review 约定

为了使评审人在评审代码时更好地专注于代码本身,请你每次提交代码时,遵守以下约定:

1)请保证CI 中测试任务能顺利通过。如果没过,说明提交的代码存在问题,评审人一般不做评审。

2)如果解决了某个Issue的问题,请在该Pull Request的第一个评论框中加上:fix #issue_number,这样当该Pull Request被合并后,会自动关闭对应的Issue。关键词包括:close, closes, closed, fix, fixes, fixed, resolve, resolves, resolved,请选择合适的词汇。详细可参考Closing issues via commit messages

此外,在回复评审人意见时,请你遵守以下约定:

1)评审人的每个意见都必须回复(这是开源社区的基本礼貌,别人帮了忙,应该说谢谢):

  • 对评审意见同意且按其修改完的,给个简单的Done即可;

  • 对评审意见不同意的,请给出你自己的反驳理由。

2)如果评审意见比较多:

  • 请给出总体的修改情况。

  • 请采用start a review进行回复,而非直接回复的方式。原因是每个回复都会发送一封邮件,会造成邮件灾难