发布于 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,它将会标记。请注意,如果你打破构建,修复通常很少,且无关紧要,所以不用担心能否通过评审周期(但不要把这个当成一个许可,即“等到构建失败时才去解决问题”)。

最新网友评论  共有(0)条评论 发布评论 返回顶部

Copyright © 2007-2017 PHPERZ.COM All Rights Reserved   冀ICP备14009818号  版权声明  广告服务