标签:很多 mit 大量 git 合并 Nging 代码重构 uav 完成
文章将了以下内容:
这个需要在时间和质量之间做选择。有的团队可能仅关注合并和主分支的功能,其他团队可能会对所有代码进行审查。
但是无论范围是什么,需要确定的是,所有人的代码都需要被复查。即使是毫无瑕疵的代码,在审查的过程中,其他人会学到很多编码的好习惯。
一般来说复查是在自测、集成测试之后,需要把代码合到主分支之前进行。
遴选
,刷新提交
等 git 功能done
review
class MyClass {
private int countTotalPageVisits; //R: name variables consistently
private int uniqueUsersCount;
}
interface MyInterface {
/** Returns {@link Optional#empty} if s cannot be extracted. */
public Optional<String> extractString(String s);
/** Returns null if {@code s} cannot be rewritten. */
//R: should harmonize return values: use Optional<> here, too
public String rewriteString(String s);
}
//R: remove and replace by Guava's MapJoiner
String joinAndConcatenate(Map<String, String> map, String keyValueSeparator, String keySeparator);
//R: This performs numIterations+1 iterations, is that intentional?
// If it is, consider changing the numIterations semantics?
for (int i = 0; i <= numIterations; ++i) {
...
}
otherService.call(); //R: I think we should avoid the dependency on OtherService. Can we discuss this in person?
标签:很多 mit 大量 git 合并 Nging 代码重构 uav 完成
原文地址:https://www.cnblogs.com/paxing/p/11296760.html