发布于 2015-12-05 10:59:27 | 193 次阅读 | 评论: 0 | 来源: 网络整理
只有提交者有提交你的新变化代码的能力,所以要确保你在代码审核中代码起作用,以使你的代码可以被提交。
如果你是一个提交者,这里有一些指南:
1.这里是列表文本检查JIRA:确保没有进一步讨论关于补丁中使用的方法。
2.遵循提交格式 :我们目前不利用任何工具来执行消息的格式,所以:
a. 提交消息是清晰和明确的。
b. 包括审查的链接(使用 support/post-reviews.py
完成自己的提交, support/apply-review.sh
完成合并他人的提交,这会使工作自动完成)。当pull了别人变化的代码 ,一定要再提交一次。
c. 使用72个字符列。注意,我们并不总是有50个字符或少总结,因为限制往往导致人们写的不好。
3.保持“作者”完整: support/apply-review.sh 将为你处理这个问题,但执行rebasing(合并) 或ammending 时要仔细。
4.不要提交合并:使用rebase合并替代,不要使用强制提交。(会覆盖他人的代码)
5.不要打破构建:我们支持Linux和Mac OS X,但是并不是所有的配置都被Jenkins 测试过,所以要意识到这一点。另外,要注意Jenkins 审查,如果重新build,它将会标记。请注意,如果你打破构建,修复通常很少,且无关紧要,所以不用担心能否通过评审周期(但不要把这个当成一个许可,即“等到构建失败时才去解决问题”)。