Blame perlcriticrc

Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
verbose = 11
Packit Service 3b7cb7
theme = (core && (pbp || bugs || maintenance || cosmetic || complexity || security || tests)) || moose
Packit Service 3b7cb7
program-extensions = pl psgi t
Packit Service 3b7cb7
Packit Service 3b7cb7
exclude = Subroutines::ProhibitCallsToUndeclaredSubs
Packit Service 3b7cb7
Packit Service 3b7cb7
[BuiltinFunctions::ProhibitStringySplit]
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[CodeLayout::RequireTrailingCommas]
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[ControlStructures::ProhibitCStyleForLoops]
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[InputOutput::RequireCheckedSyscalls]
Packit Service 3b7cb7
functions = :builtins
Packit Service 3b7cb7
exclude_functions = sleep
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[RegularExpressions::ProhibitComplexRegexes]
Packit Service 3b7cb7
max_characters = 200
Packit Service 3b7cb7
Packit Service 3b7cb7
[RegularExpressions::ProhibitUnusualDelimiters]
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[Subroutines::ProhibitUnusedPrivateSubroutines]
Packit Service 3b7cb7
private_name_regex = _(?!build)\w+
Packit Service 3b7cb7
Packit Service 3b7cb7
[TestingAndDebugging::ProhibitNoWarnings]
Packit Service 3b7cb7
allow = redefine
Packit Service 3b7cb7
Packit Service 3b7cb7
[ValuesAndExpressions::ProhibitEmptyQuotes]
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[ValuesAndExpressions::ProhibitInterpolationOfLiterals]
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[ValuesAndExpressions::RequireUpperCaseHeredocTerminator]
Packit Service 3b7cb7
severity = 3
Packit Service 3b7cb7
Packit Service 3b7cb7
[Variables::ProhibitPackageVars]
Packit Service 3b7cb7
add_packages = Carp Test::Builder
Packit Service 3b7cb7
Packit Service 3b7cb7
[-Subroutines::RequireFinalReturn]
Packit Service 3b7cb7
Packit Service 3b7cb7
# This incorrectly thinks signatures are prototypes.
Packit Service 3b7cb7
[-Subroutines::ProhibitSubroutinePrototypes]
Packit Service 3b7cb7
Packit Service 3b7cb7
[-ErrorHandling::RequireCarping]
Packit Service 3b7cb7
Packit Service 3b7cb7
# No need for /xsm everywhere
Packit Service 3b7cb7
[-RegularExpressions::RequireDotMatchAnything]
Packit Service 3b7cb7
[-RegularExpressions::RequireExtendedFormatting]
Packit Service 3b7cb7
[-RegularExpressions::RequireLineBoundaryMatching]
Packit Service 3b7cb7
Packit Service 3b7cb7
# http://stackoverflow.com/questions/2275317/why-does-perlcritic-dislike-using-shift-to-populate-subroutine-variables
Packit Service 3b7cb7
[-Subroutines::RequireArgUnpacking]
Packit Service 3b7cb7
Packit Service 3b7cb7
# "use v5.14" is more readable than "use 5.014"
Packit Service 3b7cb7
[-ValuesAndExpressions::ProhibitVersionStrings]
Packit Service 3b7cb7
Packit Service 3b7cb7
# Explicitly returning undef is a _good_ thing in many cases, since it
Packit Service 3b7cb7
# prevents very common errors when using a sub in list context to construct a
Packit Service 3b7cb7
# hash and ending up with a missing value or key.
Packit Service 3b7cb7
[-Subroutines::ProhibitExplicitReturnUndef]
Packit Service 3b7cb7
Packit Service 3b7cb7
[Variables::RequireLocalizedPunctuationVars]
Packit Service 3b7cb7
allow = %INC