Re: CPANTS new

2005-09-21 Thread Adam Kennedy
David Landgren wrote: Thomas Klausner wrote: [...] The cpants analysis fails to recognise this as valid. What is it looking for and/or could it be taught to look for this? I thought that it was only looking for a string eval of "use Test::Pod". It does, but the qq{} you're using isn't reco

Re: CPANTS new

2005-09-19 Thread David Landgren
Thomas Klausner wrote: Hi! On Sun, Sep 18, 2005 at 09:30:03PM +0200, David Landgren wrote: Yeah, but I'm loathe to dedicate two separate test files merely to score two points of Kwalitee. As it is, I'd just much rather bundle both tests in a 00_basic.t file along with all the other standard

Re: CPANTS new

2005-09-19 Thread David Landgren
Thomas Klausner wrote: [...] The cpants analysis fails to recognise this as valid. What is it looking for and/or could it be taught to look for this? I thought that it was only looking for a string eval of "use Test::Pod". It does, but the qq{} you're using isn't recognised by the regex. I'l

Re: CPANTS new

2005-09-19 Thread Thomas Klausner
Hi! On Sun, Sep 18, 2005 at 09:30:03PM +0200, David Landgren wrote: > Yeah, but I'm loathe to dedicate two separate test files merely to score > two points of Kwalitee. As it is, I'd just much rather bundle both tests > in a 00_basic.t file along with all the other standard no-brainer tests. I

Re: CPANTS new

2005-09-19 Thread Thomas Klausner
Hi! On Sun, Sep 18, 2005 at 11:48:02AM +0200, David Landgren wrote: > Seriously though, I have a module whose test suite includes Test::Pod > and Test::Pod::Coverage, except that I use the following construct: > > SKIP: { > skip( 'Test::Pod not installed on this system', 1 ) > unles

Re: CPANTS new

2005-09-19 Thread Thomas Klausner
Hi! On Sun, Sep 18, 2005 at 12:24:26PM +0200, Tels wrote: > > The cpants analysis fails to recognise this as valid. What is it > > looking for and/or could it be taught to look for this? I thought that > > it was only looking for a string eval of "use Test::Pod". > > I would like to know the sam

Re: CPANTS new

2005-09-18 Thread David Golden
David Landgren wrote: Yeah, but I'm loathe to dedicate two separate test files merely to score two points of Kwalitee. As it is, I'd just much rather bundle both tests in a 00_basic.t file along with all the other standard no-brainer tests. One option is just to forget about the two points of

Re: CPANTS new

2005-09-18 Thread David Landgren
Andrew Savige wrote: I based mine on the Test::Pod::Coverage docs: use Test::More; eval "use Test::Pod::Coverage 1.00"; plan skip_all => "Test::Pod::Coverage 1.00 required for testing POD coverage" if $@; all_pod_coverage_ok(); and scored the coverage kwalitee point... Yeah, but I'm loat

Re: CPANTS new

2005-09-18 Thread Andrew Savige
--- David Landgren wrote: > Seriously though, I have a module whose test suite includes Test::Pod > and Test::Pod::Coverage, except that I use the following construct: > > SKIP: { > skip( 'Test::Pod not installed on this system', 1 ) > unless do { > eval qq{ use Test::P

Re: CPANTS new

2005-09-18 Thread Tels
-BEGIN PGP SIGNED MESSAGE- Moin, On Sunday 18 September 2005 11:48, David Landgren wrote: > Thomas Klausner wrote: > > Hi! > > > > Data using the new metric 'has_changelog' is now available from > > http://cpants.perl.org > > Ooh! my kwalitee improved :) except other people's kwalitee imp

Re: CPANTS new

2005-09-18 Thread Tels
-BEGIN PGP SIGNED MESSAGE- Moin, On Sunday 18 September 2005 10:29, Thomas Klausner wrote: > Hi! > > Data using the new metric 'has_changelog' is now available from > http://cpants.perl.org > > Thanks again to Adam Kennedy, H.Merijn Brand and Smylers for various > suggestions/help with 'h

Re: CPANTS new

2005-09-18 Thread David Landgren
Thomas Klausner wrote: Hi! Data using the new metric 'has_changelog' is now available from http://cpants.perl.org Ooh! my kwalitee improved :) except other people's kwalitee improved more than mine :( Thanks again to Adam Kennedy, H.Merijn Brand and Smylers for various suggestions/help wi

CPANTS new

2005-09-18 Thread Thomas Klausner
Hi! Data using the new metric 'has_changelog' is now available from http://cpants.perl.org Thanks again to Adam Kennedy, H.Merijn Brand and Smylers for various suggestions/help with 'has_changelog'. I've also added suggestions to improve ones kwalitee. For each metric I wrote up a short 'remedy'