toSQL() or fromSQL() not compliant with DATETIME

[femto-code]

As stated here (https://dev.mysql.com/doc/refman/8.0/en/datetime.html) the specification for SQL requires the format (YYYY-MM-DD hh:mm:ss) to comply with the DATETIME data type.

However, luxons SQL-related functions don't seem to produce that format, is that correct? Should I go with toFormat then?

[icambron]

DateTime.now().toSQL({ includeOffset: false } ) should be what you want. The issue is only that MySQL doesn't accept the offset

[femto-code]

Alright, I see. But with the option set, it still shows milliseconds.

Is there a way to suppress that like in toISO or do I have to cut it manually?

[icambron]

Yeah, it doesn't currently support the suppressMilliseconds argument that toISO() does. My MySQL does accept milliseconds, IIRC. You can also round down to get 0s there, by calling startOf("second") on it first

[femto-code]

Okay, I agree. You can have them but I prefer cutting them off then.

Would be nice to see the suppressMilliseconds method for the toSQL function call as well though.