New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A few more tests #2

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@openstrike
Contributor

openstrike commented Feb 26, 2017

Here are a few more tests in 2 commits. The first tests contains_any() which brings the dist up to 100% subroutine coverage. The second tests the previously unexercised scenarios where the date is complete with a 4-digit year and the time is complete with seconds.

Again, this work was done as part of the CPAN PR Challenge. Thanks for participating.

@damil

This comment has been minimized.

Show comment
Hide comment
@damil

damil Mar 5, 2017

Owner

Merged. Thanks again. Hope to meet you some day at a YAPC. Cheers, L. Dami

Owner

damil commented Mar 5, 2017

Merged. Thanks again. Hope to meet you some day at a YAPC. Cheers, L. Dami

@damil damil closed this Mar 5, 2017

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