baoyachi / duration-str / 21446026744

73%

master: 73%

Build Type

Pull #89

github

Committed by baoyachi

Commit Message

feat: merge deserialize_duration and deserialize_option_duration into one (#52)

Previously, users needed to choose between deserialize_duration and
deserialize_option_duration based on whether the field was optional.

Now, a single deserialize_duration function works with both Duration
and Option<Duration> fields:

  // Required field
  #[serde(deserialize_with = deserialize_duration)]
  timeout: Duration,

  // Optional field - same function!
  #[serde(default, deserialize_with = deserialize_duration)]
  retry_delay: Option<Duration>,

Implementation:
- Introduce DeserializeDuration trait for Duration and Option<Duration>
- Use generics to dispatch to the correct implementation
- Keep old functions for backward compatibility (deprecated)

Also works for chrono::Duration and time::Duration.

Pull Request Pull Request #89: feat: merge deserialize_duration and deserialize_option_duration into…

Run Details

19 of 27 new or added lines in 1 file covered. (70.37%)

15 existing lines in 1 file now uncovered.

229 of 313 relevant lines covered (73.16%)

2.2 hits per line