time-rs / time

The most used Rust library for date and time handling.
https://time-rs.github.io
Apache License 2.0
1.09k stars 273 forks source link

Update `serde::format_description!()` docs #556

Closed TristanDebrunner closed 1 year ago

TristanDebrunner commented 1 year ago

Recent updates to the serde::format_description!() macro allow format descriptions other than format strings to be passed. This updates the documentation to reflect that.

codecov[bot] commented 1 year ago

Codecov Report

Merging #556 (d95f036) into main (0c3ceb7) will not change coverage. The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #556   +/-   ##
=====================================
  Coverage   95.7%   95.7%           
=====================================
  Files         78      78           
  Lines       8703    8703           
=====================================
  Hits        8330    8330           
  Misses       373     373           
Impacted Files Coverage Δ
.../format_description/well_known/iso8601/adt_hack.rs 100.0% <ø> (ø)
time/src/serde/mod.rs 96.5% <ø> (ø)

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

jhpratt commented 1 year ago

I added a minor commit fixing formatting (due to the way HTML handles whitespace) and correcting a typo. I also confirmed that rustfmt doesn't handle the blank lines the way I wanted — I suspect this is due to the use of #[cfg(foo, doc = "")], which doesn't interact well with formatting.

Thank you!