Add a `coverage` job to `rust.yml` using `cargo-llvm-cov` and `codecov/codecov-action`, as in apache/datafusion#23336. - [ ] Drop the unused `cargo install cargo-tarpaulin` step from `test`. - [ ] Add a `CODECOV_TOKEN` repository secret (as in apache/datafusion#23647) - [ ] Set the Codecov default branch to `main` @alamb opinions?
Activity
alamb commentedon Sep 23, 2026
My personal experience of using coverage is that they get ignored on CI personally
alamb commentedon Sep 23, 2026
That is my suggestion
LucaCappelletti94 commentedon Sep 23, 2026
I would personally make it blocking if it goes lower than the expected percentage
alamb commentedon Sep 23, 2026
I agree coverage is good-- the challenge was in getting a report i trusted and that was easy to use (coveralls doesn't seem to do it)
LucaCappelletti94 commentedon Sep 23, 2026
I agree, I stopped using coveralls a while back, personally I find codecov is much better