perf: faster sanitize date

[andrewvasilchuk]

Use String#slice instead of String#replace since it is ~20% faster in this particular case according to this performance test.

[iheanyi]

Going to close this one out. We only want to use replace in the sense that if the Z exists at the end, remove it. With this change, we're losing precision if the string doesn't contain the timezone.

[andrewvasilchuk]

@iheanyi thanks for the feedback. Following MDN Date.prototype.toISOString() and ECMAScript Language Specification, a date string, returned from the Date#toISOString method always contain a "Z" char at the end.

[iheanyi]

Good point, that's a good call then. I'll reopen this then

[ayrton]

Thank you