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

deal with more than 10 arguments #7

Merged
merged 1 commit into from Feb 1, 2017

Conversation

Projects
None yet
2 participants
@yanick
Contributor

yanick commented Jan 8, 2017

The regular expression, as it was, wouldn't deal well with 2 digit arguments (e.g., '%10' would be modified as '%1' first).

deal with more than 10 arguments
The regular expression, as it was, wouldn't deal
well with 2 digit arguments (e.g., '%10' would
be modified as '%1' first).

@ido50 ido50 merged commit 4c67719 into ido50:master Feb 1, 2017

@ido50

This comment has been minimized.

Show comment
Hide comment
@ido50

ido50 Feb 1, 2017

Owner

Hi Yanick, thank you so much for your contributions, these are really great. I've merged everything and uploaded version 1.4.0 to CPAN right now. Sorry it took my a bit long.

Thanks again, I appreciate it.

Owner

ido50 commented Feb 1, 2017

Hi Yanick, thank you so much for your contributions, these are really great. I've merged everything and uploaded version 1.4.0 to CPAN right now. Sorry it took my a bit long.

Thanks again, I appreciate it.

@yanick

This comment has been minimized.

Show comment
Hide comment
@yanick

yanick Feb 1, 2017

Contributor

My pleasure. :-) And don't worry: there was no rush there, the work was mostly for the CPAN challenge.

Contributor

yanick commented Feb 1, 2017

My pleasure. :-) And don't worry: there was no rush there, the work was mostly for the CPAN challenge.

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