[codex] fix cobertura source roots - #4
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR 背景
<sources>硬编码成src。src,例如src/main/clojure,消费者会把filename和错误的 source root 拼接,导致 diff-cover 等工具无法匹配覆盖率。PR 改了什么
:args :src-ns-path生成<sources>。filename为相对 source root 的路径。:src-ns-path缺失时,继续回退到src,保持现有 demo 行为。src/main/clojure和多 source root 场景。.cpcache/加入.gitignore,避免本地缓存污染工作区。:testalias,让新的 reporter 测试进入默认测试流程。都做了哪些测试
clj -M:test