Blame perlcriticrc

Packit a09cf7
severity = 3
Packit a09cf7
verbose = 11
Packit a09cf7
Packit a09cf7
theme = core + pbp + bugs + maintenance + cosmetic + complexity + security + tests + moose
Packit a09cf7
Packit a09cf7
exclude = Subroutines::ProhibitCallsToUndeclaredSubs
Packit a09cf7
Packit a09cf7
[BuiltinFunctions::ProhibitStringySplit]
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[CodeLayout::RequireTrailingCommas]
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[ControlStructures::ProhibitCStyleForLoops]
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[InputOutput::RequireCheckedSyscalls]
Packit a09cf7
functions = :builtins
Packit a09cf7
exclude_functions = sleep
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[Moose::RequireCleanNamespace]
Packit a09cf7
modules = Moose Moose::Role MooseX::Role::Parameterized Moose::Util::TypeConstraints
Packit a09cf7
cleaners = namespace::autoclean
Packit a09cf7
Packit a09cf7
[NamingConventions::Capitalization]
Packit a09cf7
package_exemptions = [A-Z]\w+|minFraud
Packit a09cf7
file_lexical_variables = [A-Z]\w+|[^A-Z]+
Packit a09cf7
global_variables = :starts_with_upper
Packit a09cf7
scoped_lexical_variables = [A-Z]\w+|[^A-Z]+
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
# Given our code base, leaving this at 5 would be a huge pain
Packit a09cf7
[Subroutines::ProhibitManyArgs]
Packit a09cf7
max_arguments = 10
Packit a09cf7
Packit a09cf7
[RegularExpressions::ProhibitComplexRegexes]
Packit a09cf7
max_characters = 200
Packit a09cf7
Packit a09cf7
[RegularExpressions::ProhibitUnusualDelimiters]
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[Subroutines::ProhibitUnusedPrivateSubroutines]
Packit a09cf7
private_name_regex = _(?!build)\w+
Packit a09cf7
skip_when_using = Moo::Role Moose::Role MooseX::Role::Parameterized Role::Tiny Test::Class::Moose::Role
Packit a09cf7
Packit a09cf7
[TestingAndDebugging::ProhibitNoWarnings]
Packit a09cf7
allow = redefine
Packit a09cf7
Packit a09cf7
[ValuesAndExpressions::ProhibitEmptyQuotes]
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[ValuesAndExpressions::ProhibitInterpolationOfLiterals]
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[ValuesAndExpressions::RequireUpperCaseHeredocTerminator]
Packit a09cf7
severity = 3
Packit a09cf7
Packit a09cf7
[Variables::ProhibitPackageVars]
Packit a09cf7
add_packages = Test::Builder
Packit a09cf7
Packit a09cf7
[TestingAndDebugging::RequireUseStrict]
Packit a09cf7
Packit a09cf7
[TestingAndDebugging::RequireUseWarnings]
Packit a09cf7
Packit a09cf7
[-ControlStructures::ProhibitCascadingIfElse]
Packit a09cf7
Packit a09cf7
[-ErrorHandling::RequireCarping]
Packit a09cf7
[-InputOutput::RequireBriefOpen]
Packit a09cf7
Packit a09cf7
[-ValuesAndExpressions::ProhibitConstantPragma]
Packit a09cf7
Packit a09cf7
# No need for /xsm everywhere
Packit a09cf7
[-RegularExpressions::RequireDotMatchAnything]
Packit a09cf7
[-RegularExpressions::RequireExtendedFormatting]
Packit a09cf7
[-RegularExpressions::RequireLineBoundaryMatching]
Packit a09cf7
Packit a09cf7
[-Subroutines::ProhibitExplicitReturnUndef]
Packit a09cf7
Packit a09cf7
# http://stackoverflow.com/questions/2275317/why-does-perlcritic-dislike-using-shift-to-populate-subroutine-variables
Packit a09cf7
[-Subroutines::RequireArgUnpacking]
Packit a09cf7
Packit a09cf7
[-Subroutines::RequireFinalReturn]
Packit a09cf7
Packit a09cf7
# "use v5.14" is more readable than "use 5.014"
Packit a09cf7
[-ValuesAndExpressions::ProhibitVersionStrings]