Skip to content

CI: Reduce redundant Rust checks - #2703

Open
LucaCappelletti94 wants to merge 1 commit into
ci-drop-redundant-required-checksfrom
ci-reduce-rust-check-overlap
Open

LucaCappelletti94 wants to merge 1 commit into
ci-drop-redundant-required-checksfrom
ci-reduce-rust-check-overlap

Conversation

@LucaCappelletti94

@LucaCappelletti94 LucaCappelletti94 commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

CI uses separate runner slots for compiler checks already covered by clippy. Workspace-wide linting covers the parser and benchmark targets, while line-tables-only retains 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.

@LucaCappelletti94
LucaCappelletti94 changed the base branch from main to ci-drop-redundant-required-checks October 21, 2026 12:04
@LucaCappelletti94
LucaCappelletti94 force-pushed the ci-reduce-rust-check-overlap branch from 45274ed to 3ccdceb Compare October 21, 2026 12:04
@LucaCappelletti94
LucaCappelletti94 added this pull request to stack #2705 October 21, 2026 12:11
@alamb

alamb commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

: 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.

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

@LucaCappelletti94

Copy link
Copy Markdown
Contributor Author

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-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.53%. Comparing base (155ed02) to head (3ccdceb).

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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