rt/.perlcriticrc

   1 # space separated list of strings to regex match against rules
   2 exclude = Modules::RequireFilenameMatchesPackage Subroutines::ProhibitExplicitReturnUndef Subroutines::RequireArgUnpacking
   3
   4 # [Modules::RequireFilenameMatchesPackage]
   5 # versions up to RT 3.8 use _Overlay files to separate generated code
   6 # from core code, as such there are a lot of needless warnings about
   7 # this.  These were removed in 4.0 so skipping this will not be required
   8 #
   9 # [Subroutines::ProhibitExplicitReturnUndef]
  10 # RT is documented to return undef in a number of places, and on a
  11 # stable series I'm hesitant to go around changing to implicitly return
  12 # undef.  We can clean this up in 4.0 and ensure that if something
  13 # calling us in list context really needs that explicit undef, it
  14 # doesn't break on a minor release upgrade.
  15 #
  16 # [Subroutines::RequireArgUnpacking]
  17 # RT is a big user of my $self = shift; and will continue doing so
  18 #
  19
  20 # 3.8 hasn't converted to using RT->SystemUser and the like and still reaches
  21 # for RT:: in a lot of places.  Quiet that down with what seems like a slightly
  22 # larger hammer than I would like.
  23 #
  24 # Eliminate globals declared with `our' or `use vars'.
  25 [Variables::ProhibitPackageVars]
  26 # set_themes                         = core maintenance pbp
  27 # add_themes                         =
  28 # severity                           = 3
  29 # maximum_violations_per_document    = no_limit
  30
  31 # The base set of packages to allow variables for.
  32 # packages = Data::Dumper File::Find FindBin Log::Log4perl
  33
  34 # The set of packages to allow variables for, in addition to those given in "packages".
  35 add_packages = RT
  36
  37 # Write `eval { my $foo; bar($foo) }' instead of `eval "my $foo; bar($foo);"'.
  38 [BuiltinFunctions::ProhibitStringyEval]
  39 # set_themes                         = bugs core pbp
  40 # add_themes                         =
  41 # severity                           = 5
  42 # maximum_violations_per_document    = no_limit
  43
  44 # Allow eval of "use" and "require" strings.
  45 allow_includes = 1