-
Notifications
You must be signed in to change notification settings - Fork 94
Open
Labels
type: cleanupAn internal cleanup or hygiene concern.An internal cleanup or hygiene concern.
Description
We can close this issue when we have documented why we should not do these things or we have implemented them.
With the code coverage build moving to GCB (Google Cloud Build, see #3880) it may be time to reconsider how we run this build. We use cargo tarpaulin because it was easy to set up, and was not too slow when compared to cargo llvm-cov. That may have changed, as the code size has grown.
We also exclude all the integration tests because we could not run them in GHA (GitHub Actions). That is no longer true.
In summary:
- Use
cargo llvm-covto run coverage builds. - Enable as many integration tests as possible as part of the coverage builds.
- Trim the list of directories excluded from code coverage analysis.
Metadata
Metadata
Assignees
Labels
type: cleanupAn internal cleanup or hygiene concern.An internal cleanup or hygiene concern.