Reviews by Karen Etheridge

cpanratings
 

RSS

Bundle-Git-LongList1 (0.02) *

There is absolutely zero point to this distribution. If there is, the author fails to describe it. Why do we need a Bundle that simply packages "everything"? If something wants to install all of cpan, they might as well download the PAUSE index files and iterate through them themselves.

Bundle-Git-LongList3 (0.01) *

There is absolutely zero point to this distribution. If there is, the author fails to describe it. Why do we need a Bundle that simply packages "everything"? If something wants to install all of cpan, they might as well download the PAUSE index files and iterate through them themselves.

Task-Git-LongList1 (0.03) *

There is absolutely zero point to this distribution. If there is, the author fails to describe it. Why do we need a distribution that depends on "everything"? If something wants to install all of cpan, they might as well download the PAUSE index files and iterate through them themselves.

The cpantesters infrastructure is being wasted on this distribution, testing all its dependencies (all of CPAN!) needlessly, potentially obliterating any more nuanced testing that might have been done for these distributions individually.

This distribution should either be deleted immediately, or at least the documentation should be amended to explain what this distribution's purpose is.

Bundle-Git-LongList2 (0.02) *

There is absolutely zero point to this distribution. If there is, the author fails to describe it. Why do we need a Bundle that simply packages "everything"? If something wants to install all of cpan, they might as well download the PAUSE index files and iterate through them themselves.

P (1.1.20) *

This is an excellent and comprehensive summary of everything wrong with this distribution, and the author's approach to the problem space:

ledgersmbdev.blogspot.co.uk/2013/11/o...

lambda (v0.0.1) **

This really belongs in the Acme namespace.

The-synthesizer (0.01) *

While I like Daft Punk as much as the next guy, this module doesn't belong here. "Fun" or "joke" modules should stay in the Acme:: namespace.

Test-Exception-LessClever (0.006) *

This module has outlived its usefulness -- Test::Exception has been fixed, so it is better to either use that, or use Test::Fatal which is a lighter approach to the same general problem.

This module should be deprecated and removed.

UNIVERSAL-moniker (0.08) *

Perl is not ruby. Monkey-patching UNIVERSAL is horrible. Do not use!

Bundle-Moose (0.01) **

- Does not list what version of Moose it is providing dependencies for.
- Does not list how these dependencies were determined. e.g. What does common::sense have to do with Moose? Where does this dependency come from?
- This would be better written as a generic CPAN meta-tool that traversed the dependency heirarchy for any named dist. I don't want to see an explosion of Bundle::Catalyst, Bundle::DBIx::Class, etc dists on the CPAN.

UNIVERSAL-require (0.13) **

Instead of using this, use Class::Load instead.

Hash-Work (0.03) *

General "utility libraries" such as these are not useful; this one provides harder-to-use and slower implementations than just performing native operations.

for example:

- merge_hashes{ \%hash1, \%hash2, \%hash3) just calls into Hash::Merge
- copy_hash_values is longer to type *and slower* than %newhash = %hash; as it copies every hash element separately.

Acme-SafetyGoggles (0.02) *****

Handy when playing around with various Acme modules!