Skip to content

Conversation

@moliholy
Copy link
Collaborator

@moliholy moliholy commented Oct 6, 2025

This PR:

  • Moves integration tests to a separate workflow.
  • Does not optimize disk space in the coverage workflow, as it is no longer needed due to the smaller disk consumption. Actually never mind. It is required to optimize 😓 (see the failing job).
  • Display more output in tests.
  • Updates cache names in case builds are performed with different rust flags or OS.

@moliholy moliholy requested a review from AlexD10S October 6, 2025 15:24
@moliholy moliholy self-assigned this Oct 6, 2025
@codecov
Copy link

codecov bot commented Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.33%. Comparing base (9686a6f) to head (cb74c35).
⚠️ Report is 1 commits behind head on main.

@@           Coverage Diff           @@
##             main     #653   +/-   ##
=======================================
  Coverage   77.33%   77.33%           
=======================================
  Files         109      109           
  Lines       24941    24941           
  Branches    24941    24941           
=======================================
  Hits        19288    19288           
  Misses       3678     3678           
  Partials     1975     1975           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@AlexD10S AlexD10S left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@moliholy moliholy merged commit 2cdb74b into main Oct 7, 2025
19 checks passed
@moliholy moliholy deleted the ci/optimizations branch October 7, 2025 10:07
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