Switch to llvm-cov for coverage reports

[poljar]

No description provided.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.91%. Comparing base (ee9993a) to head (d5a411c).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #213      +/-   ##
==========================================
- Coverage   90.25%   89.91%   -0.35%     
==========================================
  Files          34       34              
  Lines        1991     5383    +3392     
  Branches        0     5383    +5383     
==========================================
+ Hits         1797     4840    +3043     
- Misses        194      348     +154     
- Partials        0      195     +195     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[richvdh]

Could you give some background to this, for the archive? I gather we've been having problems with tarpaulin?

[poljar]

Could you give some background to this, for the archive? I gather we've been having problems with tarpaulin?

Sure. On the SDK side of things we saw some intermittent problems with the coverage reporting on the CI.

For a while we patched over the problem by excluding some parts of the SDK from coverage reports: matrix-org/matrix-rust-sdk#5180.

It's unclear if the SEGFAULT is caused by the added instrumentation to gather coverage data or some upstream issue.

The issue has been reported upstream but as there is not resolution yet we had to move on.

The SDK has thus switched to llvm-cov for code coverage and it only feels natural to do the same for vodozemac.

The added benefit here is that llvm-cov seems to handle incremental compilation a bit better.

[richvdh]

LGTM