-
-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Usage in CI #12
Comments
Check out the time-rs/time repo. I just added codecov using json successfully. lcov also worked. |
Actually I stand corrected. JSON does not work. lcov does. https://app.codecov.io/gh/time-rs/time |
Thanks @jhpratt. I also checked it in my test repository, it seems to work only with lcov. (Uploading of json and text reports succeeded, but an error happened in the server.) |
No problem. It looks like codecov only cares about line coverage as well, rather than the more detailed report lcov provides. But I'm pretty sure that's on codecov's end. |
Hmm. It seems codecov supports branch coverage, so that might be a rustc's problem (#8, rust-lang/rust#79649). https://docs.codecov.io/docs/about-code-coverage
llvm-cov (that used by cargo-llvm-cov internally) also supports the export of branch coverage. https://llvm.org/docs/CommandGuide/llvm-cov.html#llvm-cov-export
|
We can probably support region coverage on codecov by converting llvm-cov's json coverage format to codecov's json coverage format. |
codecov claims to support many formats, but it actually happens "There was an error processing coverage reports" errors on the server. It is necessary to understand which formats are actually supported and which ones are preferred to use.
I'm not sure about other code-coverage services as I have never used them.
The text was updated successfully, but these errors were encountered: