Skip to content

Conversation

@cakebaker
Copy link
Contributor

I noticed that while we have jobs named "cargo test" in the CI, they don't run cargo test but cargo check, and are almost identical to the "cargo check" jobs. This PR runs cargo test instead of cargo check in the "cargo test" jobs.

@codecov
Copy link

codecov bot commented May 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.80%. Comparing base (393f0b9) to head (ab1c2ba).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #392      +/-   ##
==========================================
- Coverage   58.85%   58.80%   -0.06%     
==========================================
  Files          30       30              
  Lines        3855     3855              
  Branches      846      851       +5     
==========================================
- Hits         2269     2267       -2     
  Misses       1254     1254              
- Partials      332      334       +2     

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

@sylvestre sylvestre merged commit 3badb05 into uutils:main May 28, 2024
@cakebaker cakebaker deleted the ci_run_tests branch May 28, 2024 14:52
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