Stories
Slash Boxes
Comments
NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

All the Perl that's Practical to Extract and Report

The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
 Full
 Abbreviated
 Hidden
More | Login | Reply
Loading... please wait.
  • Here's what I'd recommend doing.

    Firstly, you need a new "has_perl_version" test.

    This would check that the distribution explicitly declares (presumably in it's META.yml) the version of perl that the distribution depends on.

    Secondly, once you have that in place, make the use warnings logic something like...

    if ( dist needs 5.6+ ) {
        if ( has_warnings ) {
            return pass;
        } else {
            return fail;
        }
    } else {
        return pass;
    }
    That is, you should only fail the use warnings kwalitee metric IF you can guarantee the 100% of the userbase can handle use warnings.

    (and we've needed the has_perl_version metric for a while anyways)