|
|
|
@ -222,6 +222,7 @@ upstream
|
|
|
|
|
## 提交代码的一些约定
|
|
|
|
|
|
|
|
|
|
为了使评审人在评审代码时更好地专注于代码本身,请您每次提交代码时,遵守以下约定:
|
|
|
|
|
|
|
|
|
|
1. 请保证Travis-CI 中单元测试能顺利通过。如果没过,说明提交的代码存在问题,评审人一般不做评审。
|
|
|
|
|
2. 提交PUll Request前:
|
|
|
|
|
- 请注意commit的数量:
|
|
|
|
@ -231,6 +232,7 @@ upstream
|
|
|
|
|
3. 如果解决了某个Issue的问题,请在该PUll Request的**第一个**评论框中加上:`fix #issue_number`,这样当该PUll Request被合并后,会自动关闭对应的Issue。关键词包括:close, closes, closed, fix, fixes, fixed, resolve, resolves, resolved,请选择合适的词汇。详细可参考[Closing issues via commit messages](https://help.github.com/articles/closing-issues-via-commit-messages)。
|
|
|
|
|
|
|
|
|
|
此外,在回复评审人意见时,请您遵守以下约定:
|
|
|
|
|
|
|
|
|
|
1. 评审人的每个意见都必须回复(这是开源社区的基本礼貌,别人帮了忙,应该说谢谢):
|
|
|
|
|
- 对评审意见同意且按其修改完的,给个简单的`Done`即可;
|
|
|
|
|
- 对评审意见不同意的,请给出您自己的反驳理由。
|
|
|
|
|