summaryrefslogtreecommitdiffstats
path: root/.perlcriticrc
blob: b61acbbc53e1556f8da5b1e354ea37dc1a0dc016 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
theme = freenode || core || certrec || certrule || performance || security
severity = 1

[-BuiltinFunctions::ProhibitUselessTopic]
[-ControlStructures::ProhibitCascadingIfElse]
[-ControlStructures::ProhibitPostfixControls]
[-ControlStructures::ProhibitUnlessBlocks]
[-Documentation::RequirePodLinksIncludeText]
[-Documentation::RequirePodSections]
[-ErrorHandling::RequireCarping]
[-Modules::RequireVersionVar]
[-References::ProhibitDoubleSigils]
[-RegularExpressions::ProhibitComplexRegexes]
[-RegularExpressions::RequireExtendedFormatting]
[-Subroutines::ProhibitExcessComplexity]
[-ValuesAndExpressions::ProhibitConstantPragma]
[-ValuesAndExpressions::ProhibitEmptyQuotes]
[-ValuesAndExpressions::ProhibitMagicNumbers]
[-ValuesAndExpressions::ProhibitVersionStrings]
[-Variables::ProhibitLocalVars]
[-Variables::ProhibitPackageVars]

# this policy is broken currently
[-Freenode::PackageMatchesFilename]

# This is not a good policy.
# Non-explicit return is often easier to read.
# think of javascript arrow functions.
[-Subroutines::RequireFinalReturn]

# I don't agree with this policy because
# a bare return can actually cause more problems.
[-Subroutines::ProhibitExplicitReturnUndef]

[Variables::RequireLocalizedPunctuationVars]
allow = @ARGV $ARGV %ENV %SIG

[Variables::ProhibitPunctuationVars]
allow = $@ $! $/ $^O $^V