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

Declare required minimum Perl version #13

Merged
merged 1 commit into from Dec 7, 2018

Conversation

Projects
None yet
3 participants
@paultcochrane
Copy link
Contributor

paultcochrane commented Dec 5, 2018

Although the smartmatch tests require at least Perl 5.10, the main
modules require only Perl 5.6, hence the required version set in the
metadata is set to the lower version requirement.

I tried specifying Perl 5.10 as the required version in the test requirements in the cpanfile, unfortunately, this value then overrode the required minimum Perl version for the runtime requirements, hence I kept the minimum version at 5.6 since people should be able to install and use the module successfully with this version.

Declare required minimum Perl version
Although the smartmatch tests require at least Perl 5.10, the main
modules require only Perl 5.6, hence the required version set in the
metadata is set to the lower version requirement.

@frioux frioux merged commit 6f7dedd into frioux:master Dec 7, 2018

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
@coveralls

This comment has been minimized.

Copy link

coveralls commented Dec 8, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling d243ca3 on paultcochrane:pr/set-perl-version into 68cf9d7 on frioux:master.

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