Stop requireing Devel::Cover

[szabgab]

I think most of the people who use Test::Critic don't need the functionality provided by Devel::Cover. IMHO Test::Strict should not depend on it and if the user invokes the all_cover_ok then it should be checked if Devel::Cover is installed.

[ribasushi]

+1

if the user invokes the all_cover_ok then it should be checked if Devel::Cover is installed

This is already the case. The only part missing is this being changed to "recommends" or somesuch.

Currently Devel::Cover does not build on blead, and as such drags this module with it.

[ribasushi]

@manwar @szabgab ping: this is still a problem!

[manwar]

Hi @ribasushi,

I will sort this out by end of play today.

Best Regards,
Mohammad S Anwar

[ribasushi]

The current solution is suboptimal. While it does solve my problem of reducing the dependency footrpint, it would not alert an end-user who requested all_cover_ok that D::C is not even available. Instead it emits a diag() ( which nobody is even able to see courtesy of how cpanm operates ) and SKIP_ALLs out of the test.

You should consider failing the test when D::C is not available, and documenting that it must now be specified explicitly as a dependency.

[ribasushi]

This seems to have been resolved, @szabgab I'd close

[jkeenan]

This seems to have been resolved, @szabgab I'd close

@manwar, ^^

[manwar]

Closing the issue as confirmed above.