Repository navigation
CI: Reduce redundant Rust checks - #2703
LucaCappelletti94 wants to merge 1 commit into
Conversation
45274ed to
3ccdceb
Compare
I think the CI usage in this project is a drop in the bucket compared to other projects that share the same runner. Thus I am not sure how much effort it is worth trying to optimize our CI here much more |
Yeah I did this PR before understanding the size of the problem. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## ci-drop-redundant-required-checks #2703 +/- ##
==================================================================
Coverage 81.53% 81.53%
==================================================================
Files 42 42
Lines 34218 34218
Branches 34218 34218
==================================================================
Hits 27898 27898
Misses 2770 2770
Partials 3550 3550 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
CI uses separate runner slots for compiler checks already covered by
clippy. Workspace-wide linting covers the parser and benchmark targets, whileline-tables-onlyretains file-and-line backtraces for test failures.cc @alamb will merge this one when possible as it slims down the CI, but given we are still backlogged I am not sure what is the current cause.