timecop icon indicating copy to clipboard operation
timecop copied to clipboard

Add support for DateTime.strptime

Open thedarkone opened this issue 11 years ago • 3 comments

Full disclosure: basically, when it comes to Time/Date/DateTime, I don't really know what I'm doing.

For the implementation I copied Date's strptime code.

thedarkone avatar Mar 28 '14 16:03 thedarkone

Why is this code necessary? doesn't this work anyway?

tamird avatar Apr 04 '14 19:04 tamird

If I removed the DateTime's strptime patching, the test would fail (DateTime.strptime('04-14', '%m-%d') would use the current year instead of 1984). Am I missing something?

thedarkone avatar Apr 04 '14 21:04 thedarkone

@thedarkone i just tried your code and your test fails even with your change

travisjeffery avatar Jun 28 '14 21:06 travisjeffery