search

serde-rs / serde

Serialization framework for Rust
https://serde.rs/
Apache License 2.0
9.06k stars 767 forks source link

`IpAddr` and `SocketAddr` are in `core`, but serde implements only for `std` #2760

Closed Ddystopia closed 3 weeks ago

Ddystopia commented 3 months ago

Hello, recently IpAddr was moved to core, but serde still requires std feature

MathiasKoch commented 3 weeks ago

Would a PR fixing this be accepted, even though it would raise MSRV significantly?

oli-obk commented 3 weeks ago

This should just affect/duplicate a few imports behind cfgs, this seems fine to me.