Skip to content

Conversation

@manwar
Copy link

@manwar manwar commented Jan 17, 2017

Hi @rjbs

Please review the PR.

Many Thanks.
Best Regards,
Mohammad S Anwar

@smith153
Copy link
Collaborator

Hi,
Thanks for the PR and for the time you put into it. I'll have to check, but since Time::Piece is also included in core, I don't think the p5p maintainers like what they call "non code changes". That would include reindenting/retabbing, and certainly moving files around. Personally, I don't care, and I've thought of changing the lib directory before.

@rjbs
Copy link
Member

rjbs commented Jan 18, 2017

Time-Piece is upstream CPAN, meaning that the CPAN author can do what they like as long as they don't break things egregiously.

@scottchiefbaker
Copy link

I would like this also as it would make testing installed vs working version easier:

perl time_test.pl        # Use installed version
perl -I lib time_test.pl # Use testing version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants