Skip to content

Commit c809417

Browse files
oliverkleeoliverklee
committed
[TASK] Raise PHPStan to level 7
https://phpstan.org/user-guide/rule-levels
1 parent 0a2c875 commit c809417

2 files changed

Lines changed: 199 additions & 1 deletion

File tree

‎config/phpstan-baseline.neon‎

Lines changed: 198 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,35 @@
11
parameters:
22
ignoreErrors:
3+
-
4+
message: '#^Only booleans are allowed in an if condition, Sabberworm\\CSS\\CSSList\\CSSListItem\|false given\.$#'
5+
identifier: if.condNotBoolean
6+
count: 1
7+
path: ../src/CSSList/CSSList.php
8+
9+
-
10+
message: '#^Parameter \#1 \$animationName of method Sabberworm\\CSS\\CSSList\\KeyFrame\:\:setAnimationName\(\) expects non\-empty\-string, string given\.$#'
11+
identifier: argument.type
12+
count: 1
13+
path: ../src/CSSList/CSSList.php
14+
315
-
416
message: '#^Parameter \#2 \$found of class Sabberworm\\CSS\\Parsing\\UnexpectedTokenException constructor expects string, Sabberworm\\CSS\\Value\\CSSFunction\|Sabberworm\\CSS\\Value\\CSSString\|Sabberworm\\CSS\\Value\\LineName\|Sabberworm\\CSS\\Value\\Size\|Sabberworm\\CSS\\Value\\URL given\.$#'
517
identifier: argument.type
618
count: 1
719
path: ../src/CSSList/CSSList.php
820

21+
-
22+
message: '#^Parameter \#2 \$found of class Sabberworm\\CSS\\Parsing\\UnexpectedTokenException constructor expects string, Sabberworm\\CSS\\Value\\CSSFunction\|Sabberworm\\CSS\\Value\\LineName\|Sabberworm\\CSS\\Value\\Size\|string given\.$#'
23+
identifier: argument.type
24+
count: 1
25+
path: ../src/CSSList/CSSList.php
26+
27+
-
28+
message: '#^Parameter \#2 \$selectors2 of static method Sabberworm\\CSS\\CSSList\\CSSList\:\:selectorsMatch\(\) expects list\<Sabberworm\\CSS\\Property\\Selector\>, array\<Sabberworm\\CSS\\Property\\Selector\|string\> given\.$#'
29+
identifier: argument.type
30+
count: 1
31+
path: ../src/CSSList/CSSList.php
32+
933
-
1034
message: '#^Parameters should have "Sabberworm\\CSS\\CSSList\\CSSListItem\|array" types as the only types passed to this method$#'
1135
identifier: typePerfect.narrowPublicClassMethodParamType
@@ -18,18 +42,72 @@ parameters:
1842
count: 1
1943
path: ../src/CSSList/Document.php
2044

45+
-
46+
message: '#^Method Sabberworm\\CSS\\Comment\\Comment\:\:getShortClassName\(\) should return non\-empty\-string but returns string\.$#'
47+
identifier: return.type
48+
count: 1
49+
path: ../src/Comment/Comment.php
50+
51+
-
52+
message: '#^Method Sabberworm\\CSS\\Parsing\\ParserState\:\:parseIdentifier\(\) should return non\-empty\-string but returns string\.$#'
53+
identifier: return.type
54+
count: 1
55+
path: ../src/Parsing/ParserState.php
56+
57+
-
58+
message: '#^Method Sabberworm\\CSS\\Parsing\\ParserState\:\:strlen\(\) should return int\<0, max\> but returns int\<0, max\>\|false\.$#'
59+
identifier: return.type
60+
count: 1
61+
path: ../src/Parsing/ParserState.php
62+
2163
-
2264
message: '#^Negated boolean expression is always true\.$#'
2365
identifier: booleanNot.alwaysTrue
2466
count: 1
2567
path: ../src/Parsing/ParserState.php
2668

69+
-
70+
message: '#^Property Sabberworm\\CSS\\Parsing\\ParserState\:\:\$currentPosition \(int\<0, max\>\) does not accept int\.$#'
71+
identifier: assign.propertyType
72+
count: 2
73+
path: ../src/Parsing/ParserState.php
74+
75+
-
76+
message: '#^Parameter \#1 \$separator of class Sabberworm\\CSS\\Value\\RuleValueList constructor expects non\-empty\-string, string given\.$#'
77+
identifier: argument.type
78+
count: 1
79+
path: ../src/Rule/Rule.php
80+
81+
-
82+
message: '#^Parameter \#1 \$value of method Sabberworm\\CSS\\Rule\\Rule\:\:setValue\(\) expects Sabberworm\\CSS\\Value\\RuleValueList\|string\|null, Sabberworm\\CSS\\Value\\Value\|string given\.$#'
83+
identifier: argument.type
84+
count: 1
85+
path: ../src/Rule/Rule.php
86+
2787
-
2888
message: '#^Parameters should have "string" types as the only types passed to this method$#'
2989
identifier: typePerfect.narrowPublicClassMethodParamType
3090
count: 1
3191
path: ../src/RuleSet/DeclarationBlock.php
3292

93+
-
94+
message: '#^Property Sabberworm\\CSS\\RuleSet\\DeclarationBlock\:\:\$selectors \(list\<Sabberworm\\CSS\\Property\\Selector\>\) does not accept array\<int\<0, max\>, Sabberworm\\CSS\\Property\\Selector\>\.$#'
95+
identifier: assign.propertyType
96+
count: 1
97+
path: ../src/RuleSet/DeclarationBlock.php
98+
99+
-
100+
message: '#^Property Sabberworm\\CSS\\RuleSet\\DeclarationBlock\:\:\$selectors \(list\<Sabberworm\\CSS\\Property\\Selector\>\) does not accept list\<Sabberworm\\CSS\\Property\\Selector\|string\>\.$#'
101+
identifier: assign.propertyType
102+
count: 1
103+
path: ../src/RuleSet/DeclarationBlock.php
104+
105+
-
106+
message: '#^Parameter \#1 \$name of class Sabberworm\\CSS\\Value\\CSSFunction constructor expects non\-empty\-string, string given\.$#'
107+
identifier: argument.type
108+
count: 1
109+
path: ../src/Value/CSSFunction.php
110+
33111
-
34112
message: '#^Parameter \#2 \$arguments of class Sabberworm\\CSS\\Value\\CSSFunction constructor expects array\<Sabberworm\\CSS\\Value\\Value\|string\>\|Sabberworm\\CSS\\Value\\RuleValueList, Sabberworm\\CSS\\Value\\Value\|string given\.$#'
35113
identifier: argument.type
@@ -48,6 +126,30 @@ parameters:
48126
count: 3
49127
path: ../src/Value/Color.php
50128

129+
-
130+
message: '#^Parameter \#1 \$name of method Sabberworm\\CSS\\Value\\CSSFunction\:\:__construct\(\) expects non\-empty\-string, string given\.$#'
131+
identifier: argument.type
132+
count: 1
133+
path: ../src/Value/Color.php
134+
135+
-
136+
message: '#^Parameter \#1 \$name of method Sabberworm\\CSS\\Value\\CSSFunction\:\:setName\(\) expects non\-empty\-string, string given\.$#'
137+
identifier: argument.type
138+
count: 1
139+
path: ../src/Value/Color.php
140+
141+
-
142+
message: '#^Method Sabberworm\\CSS\\Value\\Size\:\:getShortClassName\(\) should return non\-empty\-string but returns string\.$#'
143+
identifier: return.type
144+
count: 1
145+
path: ../src/Value/Size.php
146+
147+
-
148+
message: '#^Method Sabberworm\\CSS\\Value\\Size\:\:render\(\) should return non\-empty\-string but returns string\.$#'
149+
identifier: return.type
150+
count: 1
151+
path: ../src/Value/Size.php
152+
51153
-
52154
message: '#^Parameters should have "float" types as the only types passed to this method$#'
53155
identifier: typePerfect.narrowPublicClassMethodParamType
@@ -60,6 +162,18 @@ parameters:
60162
count: 1
61163
path: ../src/Value/Size.php
62164

165+
-
166+
message: '#^Method Sabberworm\\CSS\\Value\\Value\:\:parseIdentifierOrFunction\(\) should return Sabberworm\\CSS\\Value\\CSSFunction\|string but returns Sabberworm\\CSS\\Value\\CSSFunction\|Sabberworm\\CSS\\Value\\URL\|string\.$#'
167+
identifier: return.type
168+
count: 1
169+
path: ../src/Value/Value.php
170+
171+
-
172+
message: '#^Parameter \#1 \$name of class Sabberworm\\CSS\\Value\\CSSFunction constructor expects non\-empty\-string, string given\.$#'
173+
identifier: argument.type
174+
count: 1
175+
path: ../src/Value/Value.php
176+
63177
-
64178
message: '#^Parameter \#2 \$arguments of class Sabberworm\\CSS\\Value\\CSSFunction constructor expects array\<Sabberworm\\CSS\\Value\\Value\|string\>\|Sabberworm\\CSS\\Value\\RuleValueList, Sabberworm\\CSS\\Value\\Value\|string given\.$#'
65179
identifier: argument.type
@@ -72,12 +186,48 @@ parameters:
72186
count: 1
73187
path: ../src/Value/Value.php
74188

189+
-
190+
message: '#^Method Sabberworm\\CSS\\Value\\ValueList\:\:getArrayRepresentation\(\) should return array\<string, bool\|float\|int\|list\<array\<string, mixed\>\>\|string\> but returns array\{class\: non\-empty\-string, components\: array\<array\<string, bool\|float\|int\|list\<array\<string, mixed\>\>\|string\>\>, separator\: non\-empty\-string\}\.$#'
191+
identifier: return.type
192+
count: 1
193+
path: ../src/Value/ValueList.php
194+
195+
-
196+
message: '#^Method Sabberworm\\CSS\\Value\\ValueList\:\:getShortClassName\(\) should return non\-empty\-string but returns string\.$#'
197+
identifier: return.type
198+
count: 1
199+
path: ../src/Value/ValueList.php
200+
201+
-
202+
message: '#^Parameter \#1 \$rule of class Sabberworm\\CSS\\Rule\\Rule constructor expects non\-empty\-string, string given\.$#'
203+
identifier: argument.type
204+
count: 1
205+
path: ../tests/Functional/RuleSet/RuleSetTest.php
206+
75207
-
76208
message: '#^Call to static method PHPUnit\\Framework\\Assert\:\:assertSame\(\) with ''red'' and Sabberworm\\CSS\\Value\\Value will always evaluate to false\.$#'
77209
identifier: staticMethod.impossibleType
78210
count: 1
79211
path: ../tests/ParserTest.php
80212

213+
-
214+
message: '#^Cannot call method getListComponents\(\) on Sabberworm\\CSS\\Value\\RuleValueList\|string\.$#'
215+
identifier: method.nonObject
216+
count: 2
217+
path: ../tests/ParserTest.php
218+
219+
-
220+
message: '#^Cannot call method getListSeparator\(\) on Sabberworm\\CSS\\Value\\RuleValueList\|string\.$#'
221+
identifier: method.nonObject
222+
count: 2
223+
path: ../tests/ParserTest.php
224+
225+
-
226+
message: '#^Cannot call method render\(\) on Sabberworm\\CSS\\Value\\RuleValueList\|string\.$#'
227+
identifier: method.nonObject
228+
count: 1
229+
path: ../tests/ParserTest.php
230+
81231
-
82232
message: '#^Parameter \#1 \$value of method Sabberworm\\CSS\\Rule\\Rule\:\:setValue\(\) expects Sabberworm\\CSS\\Value\\RuleValueList\|string\|null, Sabberworm\\CSS\\Value\\Size given\.$#'
83233
identifier: argument.type
@@ -107,3 +257,51 @@ parameters:
107257
identifier: argument.type
108258
count: 2
109259
path: ../tests/Unit/CSSList/CSSListTest.php
260+
261+
-
262+
message: '#^Method Sabberworm\\CSS\\Tests\\Unit\\Comment\\CommentContainerTest\:\:provideAlternativeNonemptyCommentArray\(\) should return array\<non\-empty\-string, array\{non\-empty\-list\<Sabberworm\\CSS\\Comment\\Comment\>\}\> but returns array\<non\-empty\-string, array\{list\<Sabberworm\\CSS\\Comment\\Comment\>\}\>\.$#'
263+
identifier: return.type
264+
count: 1
265+
path: ../tests/Unit/Comment/CommentContainerTest.php
266+
267+
-
268+
message: '#^Method Sabberworm\\CSS\\Tests\\Unit\\Fixtures\\ConcreteShortClassNameProvider\:\:getShortClassName\(\) should return non\-empty\-string but returns string\.$#'
269+
identifier: return.type
270+
count: 1
271+
path: ../tests/Unit/Fixtures/ConcreteShortClassNameProvider.php
272+
273+
-
274+
message: '#^Parameter \#1 \$numberOfSpaces of method Sabberworm\\CSS\\OutputFormat\:\:indentWithSpaces\(\) expects int\<1, max\>, int given\.$#'
275+
identifier: argument.type
276+
count: 1
277+
path: ../tests/Unit/OutputFormatTest.php
278+
279+
-
280+
message: '#^Parameter \#1 \$numberOfTabs of method Sabberworm\\CSS\\OutputFormat\:\:indentWithTabs\(\) expects int\<1, max\>, int given\.$#'
281+
identifier: argument.type
282+
count: 1
283+
path: ../tests/Unit/OutputFormatTest.php
284+
285+
-
286+
message: '#^Parameter \#1 \$lineNumber of method Sabberworm\\CSS\\Tests\\Unit\\Position\\Fixtures\\ConcretePosition\:\:setPosition\(\) expects int\<1, max\>\|null, int given\.$#'
287+
identifier: argument.type
288+
count: 2
289+
path: ../tests/Unit/Position/PositionTest.php
290+
291+
-
292+
message: '#^Parameter \#2 \$columnNumber of method Sabberworm\\CSS\\Tests\\Unit\\Position\\Fixtures\\ConcretePosition\:\:setPosition\(\) expects int\<0, max\>\|null, int given\.$#'
293+
identifier: argument.type
294+
count: 3
295+
path: ../tests/Unit/Position/PositionTest.php
296+
297+
-
298+
message: '#^Parameter \#1 \$rule of class Sabberworm\\CSS\\Rule\\Rule constructor expects non\-empty\-string, string given\.$#'
299+
identifier: argument.type
300+
count: 19
301+
path: ../tests/Unit/RuleSet/DeclarationBlockTest.php
302+
303+
-
304+
message: '#^Parameter \#1 \$rule of class Sabberworm\\CSS\\Rule\\Rule constructor expects non\-empty\-string, string given\.$#'
305+
identifier: argument.type
306+
count: 19
307+
path: ../tests/Unit/RuleSet/RuleSetTest.php

‎config/phpstan.neon‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ parameters:
88

99
phpVersion: 70200
1010

11-
level: 6
11+
level: 7
1212

1313
paths:
1414
- %currentWorkingDirectory%/bin/

0 commit comments

Comments
 (0)