Skip Menu |

This queue is for tickets about the Finance-Quote CPAN distribution.

Report information
The Basics
Id: 93748
Status: resolved
Worked: 5 min
Priority: 0/
Queue: Finance-Quote

People
Owner: Nobody in particular
Requestors: CHORNY [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 1.27
Fixed in: (no value)



Subject: Test::Perl::Critic should not be a dependency
Test::Perl::Critic-based test is useful only for author. Even if this tests fails, it is not a real problem, but installation still fails. Also Test::Perl::Critic has a long dependency chain, see http://deps.cpantesters.org/?module=Test%3A%3APerl%3A%3ACritic;perl=latest Usually such tests are put in xt/ directory or are run only in presence of some ENV variable. Same for Test::Kwalitee -- Alexandr Ciornii, http://chorny.net
On Tue Mar 11 03:07:45 2014, CHORNY wrote: Show quoted text
> Test::Perl::Critic-based test is useful only for author. Even if this > tests fails, it is not a real problem, but installation still fails. > Also Test::Perl::Critic has a long dependency chain, see > http://deps.cpantesters.org/?module=Test%3A%3APerl%3A%3ACritic;perl=latest > Usually such tests are put in xt/ directory or are run only in > presence of some ENV variable. > Same for Test::Kwalitee
OK. Removed them via AutoPrereqs 'skip' config of dist::zilla -- Erik