Skip Menu |

This queue is for tickets about the AppConfig CPAN distribution.

Report information
The Basics
Id: 105635
Status: open
Priority: 0/
Queue: AppConfig

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

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



Subject: Test::Pod should not be in test-requires
Test::Pod is only used by author tests; it should not be listed in TEST_REQUIRES. It looks like those pod tests can simply be moved to xt/, which will also make cpants happier (it will complain about undeclared prereqs used by t/ files, not understanding the skip_all bit.)
On 2015-07-02 14:41:14, ETHER wrote: Show quoted text
> Test::Pod is only used by author tests; it should not be listed in > TEST_REQUIRES. > > It looks like those pod tests can simply be moved to xt/, which will > also make cpants happier (it will complain about undeclared prereqs > used by t/ files, not understanding the skip_all bit.)
(I see the same issue in some of your other distributions -- I won't file tickets for all of them.) :)
On Thu Jul 02 17:52:03 2015, ETHER wrote: Show quoted text
> On 2015-07-02 14:41:14, ETHER wrote:
> > Test::Pod is only used by author tests; it should not be listed in > > TEST_REQUIRES. > > > > It looks like those pod tests can simply be moved to xt/, which will > > also make cpants happier (it will complain about undeclared prereqs > > used by t/ files, not understanding the skip_all bit.)
> > > (I see the same issue in some of your other distributions -- I won't > file tickets for all of them.) :)
https://github.com/neilb/AppConfig/pull/6
On 2016-02-15 08:58:18, ESTRABD wrote: Show quoted text
> On Thu Jul 02 17:52:03 2015, ETHER wrote:
> > On 2015-07-02 14:41:14, ETHER wrote:
> > > Test::Pod is only used by author tests; it should not be listed in > > > TEST_REQUIRES. > > > > > > It looks like those pod tests can simply be moved to xt/, which will > > > also make cpants happier (it will complain about undeclared prereqs > > > used by t/ files, not understanding the skip_all bit.)
> > > > > > (I see the same issue in some of your other distributions -- I won't > > file tickets for all of them.) :)
> > https://github.com/neilb/AppConfig/pull/6 >
I see this pull request has been merged -- could it possibly be released? AppConfig is the highest distribution on the cpan river depending on Test::Pod.