关于CodeReview中代码覆盖范围的思考

关于CodeReview中代码覆盖范围的思考

2月 19, 2020 · 1 分钟阅读时长 · 440 字 · -阅读 -评论

    CR现在的代码范围是 by单独分支,by天,代码实践中是否存在代码重看呢?

    单独按照某个分支比如sprint,我们按照提交日期筛选昨天的代码去看,这些提交实际上也是包含了MR,MR本身就是提交,而MR这种提交会包含非当天的提交记录,什么意思?就是比如你5号做的MR,可能存在4号的提交代码。

    会重复看到同一段代码吗?

    1. 日期会出现不是目标所看日期的,如上,5号的MR,包含的Diff是4号的提交
    2. 因为CR我们看的是Diff,假如之前的代码已经进入该分支,再看之后的不论哪一天,重复的代码自然不算在Diff里,所以不存在重复看同一段代码

    结论

    1. 当前机制做CR,代码覆盖范围没问题

    值得注意的

    1. MR本身是一次提交,但是MR看到的代码是MR关联的别的分支的N次commit
    2. MR本身如果存在冲突,所以MR中可能也是存在了单独在这次提交中的一些Diff修改
    3. MR这次提交日期与MR关联的commit日期是不同的,因为整体提交日期是线性的
    4. GIt记录中顺序为push日期,但记录列表我们所看到的日期是commit日期
    Alan He
    Authors
    开发者,数码产品爱好者,喜欢折腾,喜欢分享,喜欢开源