On Mon Nov 13 06:31:40 2006, ptc wrote:
> 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.
Added a check for the Perl::Critic::Bangs
On Nov 13, 2006, at 7:14 PM, Will Coleda wrote:
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
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
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.
# http://rt.perl.org/rt3/Ticket/Display.html?id=40861 >
The t/codingstd/fixme.t tes
# New Ticket Created by Paul Cochrane
# Please include the string: [perl #40861]
# in the subject line of all future correspondence about this issue.
# 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
