-
Notifications
You must be signed in to change notification settings - Fork 241
Closed
Labels
C-calendarComponent: CalendarsComponent: CalendarsT-docs-testsType: Code change outside core libraryType: Code change outside core library
Description
Currently icu_time::DateTime does not implement PartialOrd nor Ord.
Implementing the traits for it should be very simple, since Date<A> and Time already implement them - we'd only need to order by the date first and then the time.
Furthermore, ZonedDateTime also implements PartialOrd + Ord, so I don't see why DateTime couldn't.
If there are no objections to this, I'd love to make a PR for it :)
Metadata
Metadata
Assignees
Labels
C-calendarComponent: CalendarsComponent: CalendarsT-docs-testsType: Code change outside core libraryType: Code change outside core library