Skip to content

Box reform#76

Merged
simonask merged 8 commits intomainfrom
box-reform
May 3, 2025
Merged

Box reform#76
simonask merged 8 commits intomainfrom
box-reform

Conversation

@simonask
Copy link
Owner

@simonask simonask commented May 2, 2025

Closes #74

@codecov
Copy link

codecov bot commented May 2, 2025

Codecov Report

Attention: Patch coverage is 95.31250% with 24 lines in your changes missing coverage. Please review.

Project coverage is 98.98%. Comparing base (6b4281b) to head (85a059c).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/box.rs 95.31% 24 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   99.31%   98.98%   -0.33%     
==========================================
  Files          22       22              
  Lines        5946     6379     +433     
==========================================
+ Hits         5905     6314     +409     
- Misses         41       65      +24     

☔ View full report in Codecov by Sentry.
📢 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.

@simonask
Copy link
Owner Author

simonask commented May 3, 2025

(I'm taking the coverage hit - I'm exhausted and have stuff to do.)

@simonask simonask merged commit c6c6f34 into main May 3, 2025
14 of 16 checks passed
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.

Box3 has fewer methods than Box2

1 participant