Stories
Slash Boxes
Comments

All the Perl that's Practical to Extract and Report

use Perl Log In

Log In

[ Create a new account ]

kid51 (5328)

kid51
  (email not shown publicly)

Journal of kid51 (5328)

Monday June 09, 2008
06:43 PM

I vote No on proposed CPANTS metrics

[ #36636 ]
Gabor Szabo has requested comments on his proposal for addition of new metrics to CPANTS.

I vote No, for most of the same reasons already posted by chromatic.

I like Test::NoWarnings. Its author, Fergal Daly, is one of the sharpest people in the Perl QA world. But Test::Warnings is not part of the Perl core distribution. So to earn this proposed Kwalitee metric, I would have to include Test::NoWarnings underneath the t/ directory in each one of my CPAN distributions. That's pointless.

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.
  • I don't have any sympathy for the "uses_test_nowarnings" metric (mainly because it promotes Test::NoWarnings to a universal best practice to be used -- and abused -- everywhere). That happened before with Test::Pod and Test::PodCoverage and their corresponding metrics. On the other hand, to make sure that your code is free from warnings you don't know about is a good idea. But there's code that needs it and others that don't -- just like it happens with "use warnings" or "use strict".

    But you're certainl

    • Something better could be done with Module::Install and other similar developer-helper modules (which at least will let you update the embedded dependency at a later date without much suffering).

      Refusing to add unnecessary complexity which only serves to toggle one bit in a Kwalitee index is a sure sign of Actual Quality. This is one case where Kwalitee is very much at odds with Quality.