Skip to content

Conversation

ctz
Copy link
Member

@ctz ctz commented Feb 24, 2025

cargo test -- --ignored runs only ignored tests, so we've been omitting other tests in CI for a while.

`cargo test -- --ignored` runs only ignored tests, so we've
been omitting other tests in CI for a while.
Copy link

codecov bot commented Feb 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.69%. Comparing base (84b1463) to head (53a4f3c).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #325      +/-   ##
==========================================
+ Coverage   97.46%   97.69%   +0.23%     
==========================================
  Files          20       20              
  Lines        4343     4343              
==========================================
+ Hits         4233     4243      +10     
+ Misses        110      100      -10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@djc
Copy link
Member

djc commented Feb 24, 2025

Oopsie... not all that much extra coverage though.

@djc djc added this pull request to the merge queue Feb 24, 2025
Merged via the queue into main with commit 5c72344 Feb 24, 2025
64 checks passed
@djc djc deleted the jbp-run-all-tests branch February 24, 2025 12:39
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.

2 participants