Skip to content

Conversation

@jschwe
Copy link
Member

@jschwe jschwe commented Nov 26, 2025

Some unit-tests are flaky, so we rerun them normally. We didn't do this until now when measuring coverage, which has been a source of failures for the coverage job.
Additionally, we can also remove the install lld step, since we are now using a newer version of Rust, where we can just use lld out of the box.

Testing: mach try linux-coverage

Some unit-tests are flaky, so we rerun them normally.
We didn't do this until now when measuring coverage, which has been
a source of failures for the coverage job.
Additionally we can also remove the install lld step, since we are now
using a newer version of Rust, where we can just use lld out of the box.

Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>
@jschwe jschwe marked this pull request as ready for review November 26, 2025 13:28
@jschwe jschwe requested a review from sagudev as a code owner November 26, 2025 13:28
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Nov 26, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Nov 26, 2025
@jschwe jschwe added this pull request to the merge queue Nov 26, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 26, 2025
Merged via the queue into servo:main with commit 6a2e91f Nov 26, 2025
32 checks passed
@jschwe jschwe deleted the jschwender/coverage branch November 26, 2025 14:29
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants