Login
RT #72660: Exclude pragmas from Modules::RequireExplicitPackage
authorTom Wyant <harryfmudd@comcast.net>
Sat, 17 Dec 2011 23:24:12 +0000 (23:24 +0000)
committerTom Wyant <harryfmudd@comcast.net>
Sat, 17 Dec 2011 23:24:12 +0000 (23:24 +0000)
Remove the allow_perl_version configuration option that was added in
commit 4100, there having been no dissent that I am aware of since I
proposed this on the RT ticket a week ago.

Changes
lib/Perl/Critic/Policy/Modules/RequireExplicitPackage.pm
t/Modules/RequireExplicitPackage.run

diff --git a/Changes b/Changes
index d0cc266..35688fb 100644 (file)
--- a/Changes
+++ b/Changes
@@ -14,9 +14,6 @@ Next release, whenever it is:
     * Modules::RequireExplicitPackage now has configuraion option
       allow_import_of, to allow the import of specified modules before
       the package statement. RT #72660.
-    * Modules::RequireExplicitPackage now has configuraion option
-      allow_perl_version, to allow the specification of a Perl version
-      before the package statement.
     * RegularExpressions::ProhibitEnumeratedClasses no longer thinks
       that [A-Za-z_] matches \w. RT #69322.
     * RegularExpressions::ProhibitUnusedCaptures now skips the first
index 74fd23b..ac59dce 100644 (file)
@@ -37,12 +37,6 @@ sub supported_parameters {
             description    => q{Allow the specified modules to be imported outside a package},
             behavior       => 'string list',
         },
-        {
-            name           => 'allow_perl_version',
-            description    => q{Allow the specification of a Perl version outside a package},
-            default_string => '0',
-            behavior       => 'boolean',
-        },
     );
 }
 
@@ -102,9 +96,6 @@ sub _is_statement_of_interest {
         if ( my $module = $elem->module() ) {
             $self->{_allow_import_of}{$module}
                 and return $FALSE;
-        } elsif ( $elem->version() ) {
-            $self->{_allow_perl_version}
-                and return $FALSE;
         }
     }
 
@@ -171,13 +162,6 @@ following to your F<.perlcriticrc> file
 The C<allow_import_of> configuration option takes multiple module names,
 separated by spaces.
 
-It may also be desirable to allow a version of Perl to be specified
-before the package declaration, e.g. C<use 5.010;>. To allow this, add
-the following to your F<.perlcriticrc> file
-
-    [Modules::RequireExplicitPackage]
-    allow_perl_version = 1
-
 
 =head1 IMPORTANT CHANGES
 
index e1e1669..823c0c2 100644 (file)
@@ -120,17 +120,6 @@ package Foo::Bar;
 
 #-----------------------------------------------------------------------------
 
-## name Allow exception for Perl version specification
-## failures 0
-## parms { allow_perl_version => 1 }
-## cut
-
-use 5.006002;
-
-package Foo::Bar;
-
-#-----------------------------------------------------------------------------
-
 ##############################################################################
 #      $URL$
 #     $Date$