search

chronotope / chrono

Date and time library for Rust
Other
3.3k stars 523 forks source link

Turn `expect` macro into a function #1526

Closed pitdicker closed 6 months ago

pitdicker commented 6 months ago

Learned last week that our expect macro can be turned into a function if we give it a T: Copy bound.

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 50.00000% with 21 lines in your changes are missing coverage. Please review.

Project coverage is 91.83%. Comparing base (236b7ad) to head (13b4db9). Report is 1 commits behind head on main.

Files Patch % Lines
src/naive/date/mod.rs 8.33% 11 Missing :warning:
src/naive/time/mod.rs 16.66% 5 Missing :warning:
src/datetime/mod.rs 33.33% 2 Missing :warning:
src/time_delta.rs 83.33% 2 Missing :warning:
src/naive/datetime/mod.rs 66.66% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1526 +/- ## ========================================== + Coverage 91.81% 91.83% +0.01% ========================================== Files 40 40 Lines 18351 18365 +14 ========================================== + Hits 16849 16865 +16 + Misses 1502 1500 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.