|Subject:||no warning from Perl::Critic::Utils::is_script() please|
|Date:||Mon, 27 Jul 2009 11:29:59 +1000|
|To:||bug-Perl-Critic [...] rt.cpan.org|
|From:||Kevin Ryde <user42 [...] zip.com.au>|
In the current cvs I see Perl::Critic::Utils::is_script() prints a warning asking you to use ->is_script. I wish it wouldn't do that, as it makes existing policies like Lax::RequireEndWithTrueConst chatter on endlessly (a warning for every document file processed). It looks like the is_script() code is fine just dispatching to $doc->is_script, I'd hope that a note in the docs about what's now available from PPI would be enough, and keep the function and its current behaviour for compatibility.