Andy, would you accept a patch for this to optionally allow things like:
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)
On Nov 13, 2006, at 7:26 PM, Chris Dolan wrote:
On Nov 13, 2006, at 8:31 AM, Paul Cochrane (via RT) wrote:
# New Ticket Created by Paul Cochrane
# Please include the string: [perl #40861]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=40861 >
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/)
--
Will "Coke" Coleda
[EMAIL PROTECTED]
