Description
openedon Nov 15, 2022
Currently if we have a build that hits the 15 minute timeout (e.g. app-db-schema@0.1.2
) we will end up spending an hour total attempting to build the release. We do 4 builds total, each with the full timeout available to them:
- With the crates
Cargo.lock
:
a. Generate coverage data
b. Build docs - After deleting the lock:
a. Generate coverage data
b. Build docs
It would be better to only generate the coverage data if the build succeeded, so we would only attempt builds 1.b. and 2.b. before deciding it timed out, but that has issues
docs.rs/src/docbuilder/rustwide_builder.rs
Lines 658 to 660 in 2e5ef9b
One idea would be to skip the subsequent steps if one fails because of a timeout rather than a build error. It seems unlikely that unlocking the crate will turn a timeout into a successful build, or that rustdoc --show-coverage
would somehow be the cause of a timeout rather than it being one of the dependencies.