Replace chrono::DateTime

coord-e / misskey-rs

:v: Client library for Misskey :smiling_face_with_three_hearts:

Apache License 2.0

59 stars 7 forks source link

Replace chrono::DateTime #31

Open coord-e opened 2 years ago

coord-e commented 2 years ago

https://doc.rust-lang.org/std/time/struct.SystemTime.html

coord-e commented 2 years ago

Some considerations:

RFC 3339 parsing
ergonomics - maybe we should consider Into<SystemTime>

coord-e commented 2 years ago

Maybe it is ISO8601 instead of RFC3339, but in any case, the problem is not parsing but conversion to SystemTime, which may lead us to depend on time-rs anyway...

coord-e commented 2 years ago

having distinct date-time type (which does not convert to SystemTime) for ISO8601-encoded date-time field in -api and provide conversion to time::OffsetDateTime in -util would be nice

coord-e commented 2 years ago

This is no longer such a serious problem as chrono has been back in maintenance since 0.4.20.