Perl::Critic::Policy::Miscellanea::ProhibitUnrestrictedNoCritic - Forbid a bare C<## no critic>


Perl-Critic documentation  | view source Contained in the Perl-Critic distribution.

Index


NAME

Top

Perl::Critic::Policy::Miscellanea::ProhibitUnrestrictedNoCritic - Forbid a bare ## no critic

AFFILIATION

Top

This Policy is part of the core Perl::Critic distribution.

DESCRIPTION

Top

A bare ## no critic annotation will disable all the active Policies. This creates holes for other, unintended violations to appear in your code. It is better to disable only the particular Policies that you need to get around. By putting Policy names in a comma-separated list after the ## no critic annotation, then it will only disable the named Policies. Policy names are matched as regular expressions, so you can use shortened Policy names, or patterns that match several Policies. This Policy generates a violation any time that an unrestricted ## no critic annotation appears.

    ## no critic                     # not ok
    ## no critic ''                  # not ok
    ## no critic ()                  # not ok
    ## no critic qw()                # not ok

    ## no critic   (Policy1, Policy2)  # ok
    ## no critic   (Policy1 Policy2)   # ok (can use spaces to separate)
    ## no critic qw(Policy1 Policy2)   # ok (the preferred style)




NOTE

Top

Unfortunately, Perl::Critic is very sloppy about parsing the Policy names that appear after a ##no critic annotation. For example, you might be using one of these broken syntaxes...

    ## no critic Policy1 Policy2
    ## no critic 'Policy1, Policy2'
    ## no critic "Policy1, Policy2"
    ## no critic "Policy1", "Policy2"

In all of these cases, Perl::Critic will silently disable all Policies, rather than just the ones you requested. But if you use the ProhibitUnrestrictedNoCritic Policy, all of these will generate violations. That way, you can track them down and correct them to use the correct syntax, as shown above in the "DESCRIPTION". If you've been using the syntax that is shown throughout the Perl::Critic documentation for the last few years, then you should be fine.

CONFIGURATION

Top

This Policy is not configurable except for the standard options.

AUTHOR

Top

Jeffrey Ryan Thalhammer <jeff@imaginative-software.com>

COPYRIGHT

Top


Perl-Critic documentation  | view source Contained in the Perl-Critic distribution.