Skip to content

Commit 53e488b

Browse files
committed
be explicit about perl critic policies
Rather than relying on the theme and severity which are unpredictable, explicitly list the perl critic policies we want to use. Also remove all the excluded files from the critic/tidy checks, since they are all passing now.
1 parent 088cf32 commit 53e488b

File tree

2 files changed

+42
-30
lines changed

2 files changed

+42
-30
lines changed

.perlcriticrc

+42-21
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,56 @@
11
# please alpha sort config items as you add them
2-
3-
severity = 5
2+
only = 1
43
verbose = 11
5-
theme = core
6-
7-
[-ControlStructures::ProhibitPostfixControls]
8-
[-Documentation::RequirePodLinksIncludeText]
9-
[-Documentation::RequirePodSections]
10-
[-Modules::RequireVersionVar]
11-
[-RegularExpressions::RequireDotMatchAnything]
12-
[-RegularExpressions::RequireExtendedFormatting]
13-
[-RegularExpressions::RequireLineBoundaryMatching]
14-
[-Subroutines::ProhibitExplicitReturnUndef]
15-
[-Variables::ProhibitPunctuationVars]
4+
program-extensions = .t
5+
6+
[BuiltinFunctions::ProhibitSleepViaSelect]
7+
8+
[BuiltinFunctions::ProhibitStringyEval]
9+
10+
[BuiltinFunctions::RequireGlobFunction]
11+
12+
[ClassHierarchies::ProhibitOneArgBless]
1613

1714
[CodeLayout::RequireTrailingCommas]
18-
severity = 4
15+
16+
[ControlStructures::ProhibitMutatingListFunctions]
17+
18+
[InputOutput::ProhibitBarewordFileHandles]
19+
20+
[InputOutput::ProhibitInteractiveTest]
21+
22+
[InputOutput::ProhibitTwoArgOpen]
23+
24+
[InputOutput::RequireEncodingWithUTF8Layer]
25+
26+
[Modules::ProhibitEvilModules]
27+
28+
[Modules::RequireBarewordIncludes]
29+
30+
[Modules::RequireFilenameMatchesPackage]
31+
32+
[Subroutines::ProhibitNestedSubs]
33+
34+
[Subroutines::ProhibitReturnSort]
35+
36+
[Subroutines::ProhibitSubroutinePrototypes]
37+
38+
[TestingAndDebugging::RequireUseStrict]
39+
equivalent_modules = MetaCPAN::Moose
40+
41+
[TestingAndDebugging::RequireUseWarnings]
42+
equivalent_modules = MetaCPAN::Moose
1943

2044
[ValuesAndExpressions::ProhibitEmptyQuotes]
21-
severity = 4
2245

2346
[ValuesAndExpressions::ProhibitInterpolationOfLiterals]
2447
allow_if_string_contains_single_quote = 1
2548
allow = qq{} qq[]
26-
severity = 4
49+
50+
[ValuesAndExpressions::ProhibitLeadingZeros]
2751

2852
[ValuesAndExpressions::ProhibitNoisyQuotes]
29-
severity = 4
3053

31-
[TestingAndDebugging::RequireUseStrict]
32-
equivalent_modules = MetaCPAN::Moose Moose
54+
[Variables::ProhibitConditionalDeclarations]
3355

34-
[TestingAndDebugging::RequireUseWarnings]
35-
equivalent_modules = MetaCPAN::Moose Moose
56+
[Variables::RequireLexicalLoopIterators]

tidyall.ini

-9
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
[PerlTidy]
22
select = {bin,lib,t}/**/*.{pl,pm,t,psgi}
33
select = app.psgi
4-
ignore = t/encoding.t
54
argv = --profile=$ROOT/.perltidyrc
65

76
[SortLines]
@@ -15,14 +14,6 @@ ok_exit_codes = 0
1514

1615
[Test::Vars]
1716
select = {lib,t}/**/*.pm
18-
ignore = lib/MetaCPAN/Web/View/HTML.pm
1917

2018
[PerlCritic]
2119
select = {bin,lib,t}/**/*.{pl,pm,t,psgi}
22-
ignore = lib/MetaCPAN/Web/Controller/Author.pm
23-
ignore = lib/MetaCPAN/Web/Controller/Pod.pm
24-
ignore = lib/MetaCPAN/Web/Controller/Release.pm
25-
ignore = lib/MetaCPAN/Web/Model/API.pm
26-
ignore = lib/MetaCPAN/Web/Test.pm
27-
ignore = t/encoding.t
28-
ignore = t/metacpan/sitemap.t

0 commit comments

Comments
 (0)