Any plans to make this work with --archive-file ?
#265
Labels
A-nextest
Area: nextest integration https://github.com/nextest-rs/nextest
C-enhancement
Category: A new feature or an improvement for an existing one
Comments
taiki-e
added
C-enhancement
The archive contains a folder named
What do you mean with that? There is some metadata inside the archive apparently. See the following for the contents of the archive. Let me know if you need anything else or if I can support somehow by opening a PR |
|
I think it requires integrating with Maybe just put them into different archives and treat separately? AFAIU the expected behavior for testing with My use case: cargo tests are taking too long. Longer with code instrumenting. And with (cc #258) needing to run with I'd like to build tests in one job with Am I right that it's not possible now to merge coverage info created on partitions? Either as files, or reports (html, lcov, codecov et al) or sending them sequentially to i.e. codecov? |
|
If you want to do something complicated with nextest, try using show-env for now. |
|
Happy to help with machine-readable output as required! Not entirely clear to me what nextest needs to provide here, would love a summary so we can figure out some sort of API together. |
|
@sunshowers I can be a guinea pig for that. By far, following #265 (comment) I've tried to One of the goals would be to have all the following files in an archive. Though I'm not yet sure if they will be enough for the job without the entire `# cargo llvm-cov clean --workspace --verbose`UPD: just tried generating the coverage data with |
|
FWIW, I have confirmed that applying the patch mentioned in #266 (comment) to #266 works, but I have not looked into the interaction with other flags such as --partition. |
This was referenced May 3, 2023
|
I'm trying to wrap my head around this problem in my fork (and this is my first try in Rust), I wrote some observations there TriplEight#1 (comment) |
The partition does not work, that is unfortunately the problem I am also stuck with atm. Without paritioning the tests it should work fine tho EDIT: so actually this is not true. I am having problems with the archive file. For some reason nextest is not extracting everything which is necessary for llvm. |
|
Supported in v0.6.0. (#266) A simple case has been tested in CI, but if you have any problems with a more complex case, feel free to open a new issue. cargo-llvm-cov/.github/workflows/ci.yml Lines 86 to 87 in f003745 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently I am getting
Running
The text was updated successfully, but these errors were encountered: