.month() should return chrono::Month

[Chris00]

Thanks for your useful library. I was wondering why doesn't .month() return chrono::Month which can be cast to an integer if desired. To give the same result as now, Month would need to be declared as

pub enum Month {
    January = 1,
    February = 2,
    March = 3,
 ...

[djc]

Seems like an interesting idea. It would of course change the API in an incompatible way. I also think we might chose to expose the number as a method instead of relying on the integer cast?

[Chris00]

It would of course change the API in an incompatible way.

Sure. That would be for version 0.5.0. 😀

we might chose to expose the number as a method instead of relying on the integer cast?

Yes but you would need one method per integer type (.as_u8(), .as_u32(),...) while the cast takes care of that for you. The possibility to cast may be mentioned in the documentation to ease the transition.

[djc]

I think we would only provide as_u8() and leave casting it up to the user, that seems fine.