Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't allow strange leap seconds in initialization methods #1283

Merged
merged 5 commits into from
Sep 12, 2023

Deny leap second if secs != 59 in `from_num_seconds_from_midnight_opt`

8a64c8f
Select commit
Failed to load commit list.
Merged

Don't allow strange leap seconds in initialization methods #1283

Deny leap second if secs != 59 in `from_num_seconds_from_midnight_opt`
8a64c8f
Select commit
Failed to load commit list.
Codecov / codecov/project succeeded Sep 12, 2023 in 1s

91.33% (-0.09%) compared to 084b021

View this Pull Request on Codecov

91.33% (-0.09%) compared to 084b021

Details

Codecov Report

Merging #1283 (8a64c8f) into 0.4.x (084b021) will decrease coverage by 0.09%.
Report is 8 commits behind head on 0.4.x.
The diff coverage is 81.08%.

@@            Coverage Diff             @@
##            0.4.x    #1283      +/-   ##
==========================================
- Coverage   91.41%   91.33%   -0.09%     
==========================================
  Files          38       38              
  Lines       16957    16971      +14     
==========================================
- Hits        15502    15501       -1     
- Misses       1455     1470      +15     
Files Changed Coverage Δ
src/datetime/mod.rs 86.08% <ø> (-0.53%) ⬇️
src/offset/mod.rs 50.66% <ø> (ø)
src/naive/date.rs 96.30% <50.00%> (-0.05%) ⬇️
src/naive/time/mod.rs 95.95% <72.41%> (-0.56%) ⬇️
src/naive/datetime/mod.rs 97.34% <100.00%> (-0.37%) ⬇️
src/naive/time/tests.rs 98.33% <100.00%> (-0.02%) ⬇️
src/round.rs 97.72% <100.00%> (+0.14%) ⬆️

... and 4 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more