Open ytt110 opened 11 months ago
建议重新搞一个register 功能, 上报当前测试服务的服务名,分支,commit id。 后续磕一个根据来写来做数据合并。 增量代码可以用git diff + AST 来完成。
@rentiansheng 你这里为啥用到AST了呢?是要做什么?
建议重新搞一个register 功能, 上报当前测试服务的服务名,分支,commit id。 后续磕一个根据来写来做数据合并。 增量代码可以用git diff + AST 来完成。
@rentiansheng 你这里为啥用到AST了呢?是要做什么? 版本内可能会有n commit, 多次不同commit 覆盖率报告(c1, c2, c3)?
如果我在c3 的提交中,只是修改了注释?是不是用c2的覆盖率即可。
我们也遇到了这个问题,目前有什么解决方案吗?
@DaisyJiao11 这个issue中提到的 "发布多次代码",意味着代码是有变动的。而对go 覆盖率来讲,变动的代码是没法办做合并的。 所以这里所谓 "统计全量覆盖率",本质更是个业务问题,关键是怎么定义这个 "全量覆盖率"。
建议重新搞一个register 功能, 上报当前测试服务的服务名,分支,commit id。 后续磕一个根据来写来做数据合并。 增量代码可以用git diff + AST 来完成。