summaryrefslogtreecommitdiff
path: root/server/src/perl/defaultCriticProfile
blob: 0644b8cd17f2369db7b39573b15c989f7bc759a2 (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
severity=4

program-extensions = .pl .t

[TestingAndDebugging::RequireUseStrict]
equivalent_modules = Modern::Perl Object::Pad common::sense

[TestingAndDebugging::RequireUseWarnings]
equivalent_modules = Modern::Perl Object::Pad common::sense

# Gets confused with subroutine signatures
[-Subroutines::ProhibitSubroutinePrototypes]

# I like strict and warnings, but no strict and no warnings are intentional
[TestingAndDebugging::ProhibitNoStrict]
allow = vars subs refs

[-TestingAndDebugging::ProhibitNoWarnings]

# Surprisingly controversial policy, considering Community::EmptyReturn recommends the exact opposite.
[-Subroutines::ProhibitExplicitReturnUndef]

# I find code such as if($foo and !$bar) to be clear enough. 
[-ValuesAndExpressions::ProhibitMixedBooleanOperators]

# Constant is part of core and seems fairly popular
[-ValuesAndExpressions::ProhibitConstantPragma]

# Multiple packages in one file is bad form, but it's not a bug or unintentional
[-Modules::ProhibitMultiplePackages]

[-Subroutines::RequireFinalReturn]