[Bug 238230] Review Request: perl-Catalyst-Action-RenderView - Sensible default end action for view renders

bugzilla at redhat.com bugzilla at redhat.com
Wed Jun 6 03:02:22 UTC 2007


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-Catalyst-Action-RenderView - Sensible default end action for view renders


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238230





------- Additional Comments From cweyl at alumni.drew.edu  2007-06-05 23:02 EST -------
SRPM URL:
http://home.comcast.net/~ckweyl/perl-Catalyst-Action-RenderView-0.04-2.fc6.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-Catalyst-Action-RenderView.spec

I've incorporated a patch to correct the pod syntax error which caused Test::Pod
to fail.  However, pod coverage tests cannot be enabled as Test::Pod::Coverage
believes not all the documentation has been written yet:

t/03podcoverage....NOK 1                                                     
#   Failed test 'Pod coverage on Catalyst::Action::RenderView'
#   in /usr/lib/perl5/vendor_perl/5.8.8/Test/Pod/Coverage.pm at line 126.
# Coverage for Catalyst::Action::RenderView is 0.0%, with 1 naked subroutine:
#       execute
# Looks like you failed 1 test of 1.
t/03podcoverage....dubious                                                   
        Test returned status 1 (wstat 256, 0x100)
DIED. FAILED test 1
        Failed 1/1 tests, 0.00% okay

I believe execute() is called by Catalyst, not the end user, so this is not an
entirely unreasonable situation; moreover documentation is certainly provided by
the module.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.




More information about the Fedora-package-review mailing list