@ 11d698f
current
verbose = %f:%l:%c %m (%p)\n    %r\n\n

[-Bangs::ProhibitFlagComments]
[-Bangs::ProhibitNumberedNames]

[-BuiltinFunctions::ProhibitBooleanGrep]
# Boolean grep is OK since we can't use List::Util
[-BuiltinFunctions::ProhibitStringyEval]
# That's how we have to load our modules.

[-CodeLayout::ProhibitParensWithBuiltins]
[CodeLayout::ProhibitHardTabs]
allow_leading_tabs = 0
[-CodeLayout::RequireTidyCode]
# Never works for me.
[-CodeLayout::RequireTrailingCommaAtNewline]
# I like this one but it gives too many false positives.

[Compatibility::PodMinimumVersion]
above_version = 5.010001

[-ControlStructures::ProhibitCascadingIfElse]
[-ControlStructures::ProhibitCStyleForLoops]
[-ControlStructures::ProhibitDeepNests]
[-ControlStructures::ProhibitPostfixControls]
[-ControlStructures::ProhibitUnlessBlocks]

[-Documentation::PodSpelling]
[-Documentation::RequireEndBeforeLastPod]
# We could have __END__ at the end of ack if not for the squashing.
[-Documentation::RequireLinkedURLs]
[-Documentation::RequirePodAtEnd]
[-Documentation::RequirePodLinksIncludeText]
# I'm not worried about link text.
[-Documentation::RequirePodSections]


[-Editor::RequireEmacsFileVariables]

# We're a utility.  We're OK with die.
[-ErrorHandling::RequireCarping]
[-ErrorHandling::RequireUseOfExceptions]

[-InputOutput::ProhibitInteractiveTest]
[-InputOutput::ProhibitBacktickOperators]
[-InputOutput::RequireBriefOpen]
# We open files in functions and pass them around.
[-InputOutput::RequireCheckedClose]
[InputOutput::RequireCheckedSyscalls]
functions = open opendir read readline readdir

[-Modules::RequireExplicitInclusion]
# We'd really like to use this, but it's too noisy as is.
# If https://github.com/Perl-Critic/Perl-Critic-StricterSubs/issues/9 ever gets implemented, add this back in.

[-Modules::ProhibitAutomaticExportation]
[-Modules::ProhibitExcessMainComplexity]
[-Modules::ProhibitMultiplePackages]
[-Modules::RequirePerlVersion]
[-Modules::RequireVersionVar]

[-NamingConventions::Capitalization]
# Some of our variables are mixed case, like $opt_L.
[-NamingConventions::ProhibitAmbiguousNames]

[-RegularExpressions::ProhibitComplexRegexes]
[-RegularExpressions::ProhibitEnumeratedClasses]
[-RegularExpressions::ProhibitEscapedMetacharacters]
[-RegularExpressions::RequireDotMatchAnything]
[-RegularExpressions::RequireExtendedFormatting]
[-RegularExpressions::RequireLineBoundaryMatching]

[-Subroutines::ProhibitBuiltinHomonyms]
# We have plenty of funcs that we know are homonyms in ack, like open and close, but that's OK
[-Subroutines::ProhibitCallsToUndeclaredSubs]
[-Subroutines::ProhibitCallsToUnexportedSubs]
[-Subroutines::ProhibitExcessComplexity]
# Some of our subroutines are necessarily complex.
[-Subroutines::ProhibitExplicitReturnUndef]
[-Subroutines::RequireArgUnpacking]
[Subroutines::RequireFinalReturn]
terminal_funcs = die App::Ack::die

[-TestingAndDebugging::ProhibitNoStrict]
[-TestingAndDebugging::ProhibitNoWarnings]
# If I've turned off warnings or strict, I know why I'm doing it.
[-TestingAndDebugging::RequireTestLabels]

[-ValuesAndExpressions::ProhibitConstantPragma]
[-ValuesAndExpressions::ProhibitEmptyQuotes]
[-ValuesAndExpressions::ProhibitFiletest_f]
[-ValuesAndExpressions::ProhibitMagicNumbers]
[-ValuesAndExpressions::ProhibitNoisyQuotes]
[-ValuesAndExpressions::RequireConstantOnLeftSideOfEquality]
[-ValuesAndExpressions::RequireConstantVersion]
[-ValuesAndExpressions::RequireInterpolationOfMetachars]
[-ValuesAndExpressions::RequireNumericVersion]
[-ValuesAndExpressions::RestrictLongStrings]

[-Variables::ProhibitLocalVars]
[-Variables::ProhibitPackageVars]
[-Variables::ProhibitPunctuationVars]

[Variables::ProhibitReusedNames]
severity = 5
[-Variables::RequireLocalizedPunctuationVars]
# We do lots of mucking with punctuation vars and @ARGV.