Add MIN_DATETIME and MAX_DATETIME

[Kestrer]

Also add the naive versions. These would just be useful to have for completeness in the library.

[quodlibetor]

Agreed! The naive versions should be pretty trivial to add based on the impls of MIN_DATE and MAX_DATE, I'm not sure if tz-aware versions (maybe other than UTC?) should be added.

[technetos]

Hi, is it possible to get alittle more clarification on this?

[Kestrer]

@technetos something like:

// in naive
pub const MIN_TIME: NaiveTime = { secs: 0, frac: 0 };
pub const MAX_TIME: NaiveTime = { secs: u32::MAX, frac: u32::MAX };
pub const MIN_DATETIME: NaiveDateTime = { date: MIN_DATE, time: MIN_TIME };
pub const MAX_DATETIME: NaiveDateTime = { date: MAX_DATE, time: MAX_TIME };

// in root
pub const MIN_DATETIME: DateTime<Utc> = { datetime: naive::MIN_DATETIME, offset: Utc };
pub const MAX_DATETIME: DateTime<Utc> = { datetime: naive::MAX_DATETIME, offset: Utc };

Perhaps MIN_UTC_DATETIME is a better name, or you could even use associated trait consts for TimeZone (although I don't know if that's possible).

[LAC-Tech]

Hi, while I can confirm these are now in the code, they're not in the docs (Came in here to ask about them!).

[djc]

Where are you looking? I can find them in the docs. Note: these are not associated constants, they're just bare constants in for example the naive module.

[LAC-Tech]

https://docs.rs/chrono/0.4.0/chrono/index.html#constants

Pretty much here. I'm pretty new to rust, so very possible this is my issue, but I can't find them documented anywhere.

[djc]

Note this link:

You're looking at documentation for 0.4.0, which is almost 5 years old at this point.

[rikvdkleij]

The issue is that MIN_TIME and MAX_TIME of NaiveTime are not exposed by re-exporting while the other ones are.

See

chrono/src/lib.rs

Lines 475 to 490 in 13e1d48

	pub use date::{Date, MAX_DATE, MIN_DATE};
	mod datetime;
	#[cfg(feature = "rustc-serialize")]
	pub use datetime::rustc_serialize::TsSeconds;
	pub use datetime::{DateTime, SecondsFormat, MAX_DATETIME, MIN_DATETIME};
	pub mod format;
	/// L10n locales.
	#[cfg(feature = "unstable-locales")]
	pub use format::Locale;
	pub use format::{ParseError, ParseResult};
	pub mod naive;
	#[doc(no_inline)]
	pub use naive::{IsoWeek, NaiveDate, NaiveDateTime, NaiveTime, NaiveWeek};

[djc]

Feel free to submit a PR fixing that.

[rikvdkleij]

@djc Done, see #713