Skip to content
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

Propose patch for CPANTESTER fail report. #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

manwar
Copy link

@manwar manwar commented Oct 28, 2018

Hi @kfly8

Please review the PR.

The use logical or operator // was introduced in Perl v5.10. There we see FAIL reports on platform having Perl v5.8. So the simple fix is to upgrade the minimum perl requirement to v5.10.

    https://www.cpantesters.org/cpan/report/308d01ea-d763-11e7-bba1-b275c4fe1824
    https://www.cpantesters.org/cpan/report/28fca39c-dd29-11e7-a1cf-bb670eaac09d

Many Thanks.
Best Regards,
Mohammad S Anwar

The use logical or operator // was introduced in Perl v5.10. There we see FAIL
reports on platform having Perl v5.8. So the simple fix is to upgrade the minimum
perl requirement to v5.10.

  https://www.cpantesters.org/cpan/report/308d01ea-d763-11e7-bba1-b275c4fe1824
  https://www.cpantesters.org/cpan/report/28fca39c-dd29-11e7-a1cf-bb670eaac09d
@kfly8
Copy link
Owner

kfly8 commented Nov 3, 2018

Thank you for your pull request.

I will change operator //.

kfly8 added a commit that referenced this pull request Dec 31, 2018
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.

2 participants