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

Added minimum value for perl to Prereqs in dist.ini #1

Merged
merged 1 commit into from Jan 21, 2015

Conversation

Projects
None yet
2 participants
@herveus
Contributor

herveus commented Jan 16, 2015

perlver claimed 5.6.0 was sufficient

I'm doing the CPAN challenge and was assigned this module. I'm making a minimal change that will increase the Kwalitee score to get my feet wet. If there is something more useful that I might do, I'm open to suggestions for things to look at. Thanks.

Added minimum value for perl to Prereqs in dist.ini
perlver claimed 5.6.0 was sufficient
@jcop

This comment has been minimized.

Show comment
Hide comment
@jcop

jcop Jan 21, 2015

Collaborator

Sorry for the delay in answering.
I'm currently changing the code base, rewriting old Perl 4 style snippets and reviewing the tests and documentation.
I suspect that some of the prerequisites make no sense, and others are missing...
I'm also trying to automate large part of the management process with dzil and a few plugins.

In any case, I'll merge in your change.
Thanks for the effort! :)

Collaborator

jcop commented Jan 21, 2015

Sorry for the delay in answering.
I'm currently changing the code base, rewriting old Perl 4 style snippets and reviewing the tests and documentation.
I suspect that some of the prerequisites make no sense, and others are missing...
I'm also trying to automate large part of the management process with dzil and a few plugins.

In any case, I'll merge in your change.
Thanks for the effort! :)

jcop added a commit that referenced this pull request Jan 21, 2015

Merge pull request #1 from herveus/minver
Added minimum value for perl to Prereqs in dist.ini

@jcop jcop merged commit db38996 into tavaresb:master Jan 21, 2015

@herveus herveus deleted the herveus:minver branch Feb 24, 2015

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