Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Commit e9ee22f

Browse files
Fix deser/ser impls in merde_time after phasing out JsonSerialize trait
1 parent 1b7633d commit e9ee22f

File tree

3 files changed

+7
-11
lines changed

3 files changed

+7
-11
lines changed

merde/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ serde = ["merde_core/serde"]
8585
rusqlite = ["merde_core/rusqlite"]
8686

8787
# non-core crates
88-
json = ["dep:merde_json", "merde_time/json"]
88+
json = ["dep:merde_json"]
8989
yaml = ["dep:merde_yaml"]
9090
msgpack = ["dep:merde_msgpack"]
9191
time = ["dep:merde_time"]

merde_time/Cargo.toml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,13 @@ merde_json = { version = "8.0.1", path = "../merde_json", optional = true }
1616
time = "0.3.36"
1717

1818
[dev-dependencies]
19+
merde_json = { path = "../merde_json" }
1920
time = { version = "0.3.36", features = ["macros"] }
2021

2122
[features]
2223
default = []
23-
full = ["merde", "json", "serialize", "deserialize"]
24+
full = ["merde", "serialize", "deserialize"]
2425
merde = ["dep:merde_core"]
25-
json = ["merde", "dep:merde_json", "serialize"]
26+
json = []
2627
serialize = ["time/formatting"]
2728
deserialize = ["merde", "time/parsing"]

merde_time/src/lib.rs

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
//! Provides [Rfc3339], a wrapper around [time::OffsetDateTime] that implements
2-
//! [merde_json::JsonSerialize] and [merde_core::Deserialize] when the right
2+
//! [merde_core::Serialize] and [merde_core::Deserialize] when the right
33
//! cargo features are enabled.
44
55
use std::{
@@ -9,7 +9,7 @@ use std::{
99

1010
pub use time::OffsetDateTime;
1111

12-
/// A wrapper around date-time types that implements `JsonSerialize` and `JsonDeserialize`
12+
/// A wrapper around date-time types that implements `Serialize` and `Deserialize`
1313
/// when the right cargo features are enabled.
1414
#[derive(Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]
1515
#[repr(transparent)]
@@ -81,11 +81,6 @@ mod merde_impls {
8181
))
8282
}
8383
}
84-
}
85-
86-
#[cfg(feature = "json")]
87-
mod merde_json_impls {
88-
use super::*;
8984

9085
#[cfg(feature = "serialize")]
9186
impl merde_core::Serialize for Rfc3339<time::OffsetDateTime> {
@@ -104,7 +99,7 @@ mod merde_json_impls {
10499
}
105100
}
106101

107-
#[cfg(all(test, feature = "full",))]
102+
#[cfg(all(test, feature = "full"))]
108103
mod tests {
109104
use super::*;
110105
use merde_json::{from_str, JsonSerialize};

0 commit comments

Comments
 (0)