The t/codingstd/fixme.t test only looks in C-language files for
FIXME|TODO|XXX comments. It would therefore be good for the
t/codingstd/perlcritic.t test to check for these comments in the Perl
source files.
There is a policy on CPAN to do this:
http://search.cpan.org/~petdance/Perl-Critic-Bangs-0.22/
ProhibitFlagComments.pm
Chris
--
Chris Dolan, Software Developer, Clotho Advanced Media Inc.
608-294-7900, fax 294-7025, 1435 E Main St, Madison WI 53703
vCard: http://www.chrisdolan.net/ChrisDolan.vcf
Clotho Advanced Media, Inc. - Creators of MediaLandscape Software
(http://www.media-landscape.com/) and partners in the revolutionary
Croquet project (http://www.opencroquet.org/)
FIXME (#40123)
but not:
FIXME
Which would allow us to let this slide if someone already opened a
ticket on it? (Probably want to specify the option as an RE)
--
Will "Coke" Coleda
wi...@coleda.com
> Andy, would you accept a patch for this to optionally allow things
> like:
>
> FIXME (#40123)
No, I don't want to build in exceptions.
However, how about if the RT tickets are noted as "RT #40123", which
is just as easy to find?
--
Andy Lester => an...@petdance.com => www.petdance.com => AIM:petdance
Added a check for the Perl::Critic::Bangs module into t/codingstd/
perlcritic.t and if it is installed, the Bangs::ProhibitFlagComments
policy is added to the list of policies. This change was made in
r17925. All FIXME|TODO|XXX comments which have been added to RT should
be replaced with RT#<ticket number>.
Paul