RecordStream icon indicating copy to clipboard operation
RecordStream copied to clipboard

Published 20 hours ago • verified publisher icon benbernard

parsedate: Parse and reformat dates and times

Open tsibley opened this issue 9 years ago • 6 comments

Basically a front-end to Time::ParseDate, which offers more control than Date::Manip::DM5's ParseDate(). The fatpack recs, already 7.4MB, is increased only 28KB by Time::ParseDate, so there's no huge loss to avoid it. POSIX, luckily, is part of core.

tsibley avatar Feb 25 '16 06:02 tsibley

Yay! Travis caught a problem with older Perls I didn't realize would be an issue. I'll fix it up tomorrow.

tsibley avatar Feb 25 '16 06:02 tsibley

This branch is now good to go. @benbernard, would you take a look?

tsibley avatar Feb 26 '16 05:02 tsibley

Looks great! I had a few comments, but feel free to merge as-is

benbernard avatar Feb 26 '16 17:02 benbernard

Thanks for the notes! I'll make the changes you recommended and merge away.

tsibley avatar Feb 28 '16 19:02 tsibley

In adding some tests, I just found a bug (timezones, naturally), so I'm going to fix that and repush at some point (probably not today).

tsibley avatar Feb 28 '16 19:02 tsibley

awesome, glad you found it!

On Sun, Feb 28, 2016 at 11:53 AM Thomas Sibley [email protected] wrote:

In adding some tests, I just found a bug (timezones, naturally), so I'm going to fix that and repush at some point (probably not today).

— Reply to this email directly or view it on GitHub https://github.com/benbernard/RecordStream/pull/74#issuecomment-189933597 .

benbernard avatar Feb 29 '16 19:02 benbernard