fdeantoni/prost-wkt

PartialOrd and Ord for `Timestamp`

bjchambers opened this issue · 1 comments

It seems like Timestamp could (and perhaps should) implement PartialOrd and Ord. This should be pretty straightforward given the constraints -- compare the seconds, and if equal compare the nanos. Any reason these shouldn't be added?

This project simply takes implementations of Timestamp from Prost-types, so I'm hesitant to define it here. Perhaps it should be raised as an issue in the tokio-rs/prost project instead?