Skip to content
This repository was archived by the owner on Jan 3, 2023. It is now read-only.

Commit 0baa211

Browse files
janlangerjanlanger
committed
Remove consistence/coding-standard
1 parent 1e8e310 commit 0baa211

File tree

3 files changed

+272
-28
lines changed

3 files changed

+272
-28
lines changed

‎.github/workflows/build.yml‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ jobs:
4646

4747
- name: "Install dependencies PHP 8"
4848
if: matrix.php-version == '8.0'
49-
run: "composer update --prefer-dist --no-interaction --no-progress --no-suggest --ignore-platform-req=php"
49+
run: "composer update --prefer-dist --no-interaction --no-progress --no-suggest"
5050

5151
- name: "Lint"
5252
run: "vendor/bin/phing check-phplint"
@@ -118,7 +118,7 @@ jobs:
118118

119119
- name: "Install dependencies PHP 8"
120120
if: matrix.php-version == '8.0'
121-
run: "composer update --prefer-dist --no-interaction --no-progress --no-suggest --ignore-platform-req=php"
121+
run: "composer update --prefer-dist --no-interaction --no-progress --no-suggest"
122122

123123
- name: "PHPStan"
124124
run: "vendor/bin/phing check-phpstan"
@@ -167,15 +167,15 @@ jobs:
167167

168168
- name: "Install lowest dependencies PHP 8"
169169
if: ${{ matrix.php-version == '8.0' && matrix.dependencies == 'lowest' }}
170-
run: "composer update --prefer-lowest --prefer-dist --no-interaction --no-progress --no-suggest --ignore-platform-req=php"
170+
run: "composer update --prefer-lowest --prefer-dist --no-interaction --no-progress --no-suggest"
171171

172172
- name: "Require higher phpunit on PHP 8"
173173
if: ${{ matrix.php-version == '8.0' && matrix.dependencies == 'lowest' }}
174-
run: "composer require phpunit/phpunit:^9.3.7 --update-with-dependencies --prefer-dist --no-interaction --no-progress --no-suggest --ignore-platform-req=php"
174+
run: "composer require phpunit/phpunit:^9.3.7 --update-with-dependencies --prefer-dist --no-interaction --no-progress --no-suggest"
175175

176176
- name: "Install highest dependencies PHP 8"
177177
if: ${{ matrix.php-version == '8.0' && matrix.dependencies == 'highest' }}
178-
run: "composer update --prefer-dist --no-interaction --no-progress --no-suggest --ignore-platform-req=php"
178+
run: "composer update --prefer-dist --no-interaction --no-progress --no-suggest"
179179

180180
- name: "Tests"
181181
run: "vendor/bin/phing check-tests"

‎build/ruleset.xml‎

Lines changed: 267 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -5,26 +5,277 @@
55
<arg value="s"/>
66
<arg value="p"/>
77

8-
<rule ref="./vendor/consistence/coding-standard/Consistence/ruleset.xml">
9-
<exclude name="Consistence.Exceptions.ExceptionDeclaration.IncorrectExceptionDirectory"/>
10-
<exclude name="Consistence.Exceptions.ExceptionDeclaration.NotChainable"/>
11-
<exclude name="Generic.Arrays.ArrayIndent"/><!-- buggy -->
8+
<rule ref="Generic.Arrays.DisallowLongArraySyntax"/>
9+
<rule ref="Generic.Classes.DuplicateClassName"/>
10+
<rule ref="Generic.CodeAnalysis.EmptyStatement">
11+
<exclude name="Generic.CodeAnalysis.EmptyStatement.DetectedCatch"/><!-- empty catch statements are allowed -->
1212
<exclude name="Generic.CodeAnalysis.EmptyStatement.DetectedIf"/><!-- allow empty if statements - usually with a comment -->
1313
<exclude name="Generic.CodeAnalysis.EmptyStatement.DetectedElse"/><!-- allow empty else statements - usually with a comment -->
1414
<exclude name="Generic.CodeAnalysis.EmptyStatement.DetectedElseif"/><!-- allow empty elseif statements - usually with a comment -->
15-
<exclude name="PEAR.WhiteSpace.ObjectOperatorIndent.Incorrect"/><!-- je potřeba odsazení u formů a dibi fluent -->
16-
<exclude name="PSR2.Methods.FunctionCallSignature.CloseBracketLine"/><!-- string rozdělený na více řádků, typicky SQL -->
17-
<exclude name="PSR2.Methods.FunctionCallSignature.MultipleArguments"/><!-- dibi syntaxe využívá argumentů "v textu" -->
18-
<exclude name="PSR2.Methods.FunctionCallSignature.SpaceBeforeCloseBracket"/><!-- conflicts with our way of writing multiline SQL queries -->
19-
<exclude name="SlevomatCodingStandard.Namespaces.FullyQualifiedClassNameAfterKeyword"/>
20-
<exclude name="SlevomatCodingStandard.Namespaces.FullyQualifiedClassNameInAnnotation"/>
21-
<exclude name="SlevomatCodingStandard.Namespaces.FullyQualifiedExceptions"/>
22-
<exclude name="Squiz.Commenting.FunctionComment.InvalidNoReturn"/> <!-- does not recognize method that only throws exception -->
23-
<exclude name="SlevomatCodingStandard.Numbers.DisallowNumericLiteralSeparator"/>
24-
<exclude name="SlevomatCodingStandard.Numbers.RequireNumericLiteralSeparator"/>
25-
<exclude name="SlevomatCodingStandard.PHP.DisallowReference"/>
26-
<exclude name="SlevomatCodingStandard.PHP.RequireExplicitAssertion"/>
2715
</rule>
16+
<rule ref="Generic.CodeAnalysis.ForLoopShouldBeWhileLoop"/>
17+
<rule ref="Generic.CodeAnalysis.UnconditionalIfStatement"/>
18+
<rule ref="Generic.CodeAnalysis.UnnecessaryFinalModifier"/>
19+
<rule ref="Generic.ControlStructures.InlineControlStructure"/>
20+
<rule ref="Generic.Files.ByteOrderMark"/>
21+
<rule ref="Generic.Files.InlineHTML"/>
22+
<rule ref="Generic.Files.LineEndings">
23+
<properties>
24+
<property name="eolChar" value="\n"/>
25+
</properties>
26+
</rule>
27+
<rule ref="Generic.Formatting.DisallowMultipleStatements"/>
28+
<rule ref="Generic.Formatting.SpaceAfterCast"/>
29+
<rule ref="Generic.Functions.FunctionCallArgumentSpacing"/>
30+
<rule ref="Generic.Functions.OpeningFunctionBraceBsdAllman"/>
31+
<rule ref="Generic.NamingConventions.ConstructorName"/>
32+
<rule ref="Generic.NamingConventions.UpperCaseConstantName"/>
33+
<rule ref="Generic.PHP.CharacterBeforePHPOpeningTag"/>
34+
<rule ref="Generic.PHP.DeprecatedFunctions"/>
35+
<rule ref="Generic.PHP.DisallowShortOpenTag"/>
36+
<rule ref="Generic.PHP.ForbiddenFunctions"/>
37+
<rule ref="Generic.PHP.LowerCaseConstant"/>
38+
<rule ref="Generic.PHP.LowerCaseKeyword"/>
39+
<rule ref="Generic.PHP.LowerCaseType"/>
40+
<rule ref="Generic.Strings.UnnecessaryStringConcat">
41+
<properties>
42+
<property name="allowMultiline" value="true"/>
43+
</properties>
44+
</rule>
45+
<rule ref="Generic.WhiteSpace.ArbitraryParenthesesSpacing">
46+
<properties>
47+
<property name="ignoreNewlines" value="true"/>
48+
</properties>
49+
</rule>
50+
<rule ref="Generic.WhiteSpace.DisallowSpaceIndent"/>
51+
<rule ref="Generic.WhiteSpace.LanguageConstructSpacing"/>
52+
<rule ref="Generic.WhiteSpace.ScopeIndent">
53+
<properties>
54+
<property name="tabIndent" value="true"/>
55+
</properties>
56+
</rule>
57+
<rule ref="PEAR.Classes.ClassDeclaration"/>
58+
<rule ref="PEAR.Commenting.InlineComment"/>
59+
<rule ref="PEAR.Formatting.MultiLineAssignment"/>
60+
<rule ref="PEAR.WhiteSpace.ObjectOperatorIndent"/>
61+
<rule ref="PSR1.Classes.ClassDeclaration"/>
62+
<rule ref="PSR1.Files.SideEffects"/>
63+
<rule ref="PSR1.Methods.CamelCapsMethodName"/>
64+
<rule ref="PSR2.Classes.PropertyDeclaration"/>
65+
<rule ref="PSR2.ControlStructures.ControlStructureSpacing">
66+
<exclude name="PSR2.ControlStructures.ControlStructureSpacing.SpacingAfterOpenBrace"/><!-- we want to put first expression of multiline condition on next line -->
67+
</rule>
68+
<rule ref="PSR2.ControlStructures.ElseIfDeclaration"/>
69+
<rule ref="PSR2.ControlStructures.SwitchDeclaration">
70+
<exclude name="PSR2.ControlStructures.SwitchDeclaration.caseIndent"/><!-- checked by more generic Generic.WhiteSpace.ScopeIndent.Incorrect -->
71+
<exclude name="PSR2.ControlStructures.SwitchDeclaration.defaultIndent"/><!-- checked by more generic Generic.WhiteSpace.ScopeIndent.Incorrect -->
72+
</rule>
73+
<rule ref="PSR2.Files.ClosingTag"/>
74+
<rule ref="PSR2.Files.EndFileNewline"/>
75+
<rule ref="PSR2.Methods.FunctionCallSignature">
76+
<exclude name="PSR2.Methods.FunctionCallSignature.SpaceAfterCloseBracket"/><!-- space after closing bracked should be checked depending on where function is called, not as part of the function call -->
77+
</rule>
78+
<rule ref="PSR2.Methods.FunctionClosingBrace"/>
79+
<rule ref="PSR2.Methods.MethodDeclaration"/>
80+
<rule ref="PSR2.Namespaces.UseDeclaration"/>
81+
<rule ref="SlevomatCodingStandard.Arrays.TrailingArrayComma"/>
82+
<rule ref="SlevomatCodingStandard.Classes.ClassConstantVisibility"/>
83+
<rule ref="SlevomatCodingStandard.Classes.EmptyLinesAroundClassBraces"/>
84+
<rule ref="SlevomatCodingStandard.Classes.UnusedPrivateElements"/>
85+
<rule ref="SlevomatCodingStandard.Commenting.EmptyComment"/>
86+
<rule ref="SlevomatCodingStandard.Commenting.InlineDocCommentDeclaration"/>
87+
<rule ref="SlevomatCodingStandard.Commenting.RequireOneLinePropertyDocComment"/>
88+
<rule ref="SlevomatCodingStandard.Commenting.UselessFunctionDocComment">
89+
<properties>
90+
<property name="traversableTypeHints" type="array">
91+
<element value="Traversable"/>
92+
</property>
93+
</properties>
94+
</rule>
95+
<rule ref="SlevomatCodingStandard.Commenting.UselessInheritDocComment"/>
96+
<rule ref="SlevomatCodingStandard.ControlStructures.DisallowEmpty"/>
97+
<rule ref="SlevomatCodingStandard.ControlStructures.DisallowYodaComparison"/>
98+
<rule ref="SlevomatCodingStandard.ControlStructures.LanguageConstructWithParentheses"/>
99+
<rule ref="SlevomatCodingStandard.ControlStructures.NewWithParentheses"/>
100+
<rule ref="SlevomatCodingStandard.Exceptions.DeadCatch"/>
101+
<rule ref="SlevomatCodingStandard.Exceptions.ReferenceThrowableOnly"/>
102+
<rule ref="SlevomatCodingStandard.PHP.ShortList"/>
103+
<rule ref="SlevomatCodingStandard.PHP.TypeCast"/>
104+
<rule ref="SlevomatCodingStandard.Namespaces.AlphabeticallySortedUses">
105+
<properties>
106+
<property name="caseSensitive" value="true"/>
107+
<property name="psr12Compatible" value="false"/>
108+
</properties>
109+
</rule>
110+
<rule ref="SlevomatCodingStandard.Namespaces.DisallowGroupUse"/>
111+
<rule ref="SlevomatCodingStandard.Namespaces.MultipleUsesPerLine"/>
112+
<rule ref="SlevomatCodingStandard.Namespaces.NamespaceDeclaration"/>
113+
<rule ref="SlevomatCodingStandard.Namespaces.NamespaceSpacing"/>
114+
<rule ref="SlevomatCodingStandard.Namespaces.ReferenceUsedNamesOnly">
115+
<properties>
116+
<property name="allowFullyQualifiedExceptions" value="false"/>
117+
<property name="allowPartialUses" value="false"/>
118+
</properties>
119+
</rule>
120+
<rule ref="SlevomatCodingStandard.Namespaces.RequireOneNamespaceInFile"/>
121+
<rule ref="SlevomatCodingStandard.Namespaces.UnusedUses">
122+
<properties>
123+
<property name="searchAnnotations" value="true"/>
124+
</properties>
125+
</rule>
126+
<rule ref="SlevomatCodingStandard.Namespaces.UseDoesNotStartWithBackslash"/>
127+
<rule ref="SlevomatCodingStandard.Namespaces.UseFromSameNamespace"/>
128+
<rule ref="SlevomatCodingStandard.TypeHints.DeclareStrictTypes">
129+
<properties>
130+
<property name="newlinesCountBetweenOpenTagAndDeclare" value="2"/>
131+
</properties>
132+
</rule>
133+
<rule ref="SlevomatCodingStandard.TypeHints.LongTypeHints"/>
134+
<rule ref="SlevomatCodingStandard.TypeHints.NullableTypeForNullDefaultValue"/>
135+
<rule ref="SlevomatCodingStandard.TypeHints.ParameterTypeHintSpacing"/>
136+
<rule ref="SlevomatCodingStandard.TypeHints.ParameterTypeHint">
137+
<exclude name="SlevomatCodingStandard.TypeHints.ParameterTypeHint.UselessAnnotation"/>
138+
<properties>
139+
<property name="traversableTypeHints" type="array">
140+
<element value="Traversable"/>
141+
</property>
142+
</properties>
143+
</rule>
144+
<rule ref="SlevomatCodingStandard.TypeHints.PropertyTypeHint">
145+
<properties>
146+
<property name="traversableTypeHints" type="array">
147+
<element value="Traversable"/>
148+
</property>
149+
</properties>
150+
</rule>
151+
<rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHint">
152+
<exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.UselessAnnotation"/>
153+
<properties>
154+
<property name="traversableTypeHints" type="array">
155+
<element value="Traversable"/>
156+
</property>
157+
</properties>
158+
</rule>
159+
<rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHintSpacing"/>
160+
<rule ref="SlevomatCodingStandard.TypeHints.UselessConstantTypeHint"/>
161+
<rule ref="Squiz.Arrays.ArrayBracketSpacing">
162+
<exclude name="Squiz.Arrays.ArrayBracketSpacing.SpaceBeforeBracket"/><!-- does not handle nested array access across multiple lines -->
163+
</rule>
164+
<rule ref="Squiz.Arrays.ArrayDeclaration">
165+
<exclude name="Squiz.Arrays.ArrayDeclaration.CloseBraceNewLine"/><!-- does not handle wrapped content -->
166+
<exclude name="Squiz.Arrays.ArrayDeclaration.CloseBraceNotAligned"/><!-- expects closing brace at the same level as opening brace -->
167+
<exclude name="Squiz.Arrays.ArrayDeclaration.DoubleArrowNotAligned"/><!-- we don't want spacing with alignment -->
168+
<exclude name="Squiz.Arrays.ArrayDeclaration.FirstIndexNoNewline"/><!-- expects multi-value array always written on multiple lines -->
169+
<exclude name="Squiz.Arrays.ArrayDeclaration.FirstValueNoNewline"/><!-- expects multi-value array always written on multiple lines -->
170+
<exclude name="Squiz.Arrays.ArrayDeclaration.IndexNoNewline"/><!-- false positives with nested arrays https://github.com/squizlabs/PHP_CodeSniffer/issues/2937#issuecomment-615262067 -->
171+
<exclude name="Squiz.Arrays.ArrayDeclaration.KeyNotAligned"/><!-- uses indentation of only single space -->
172+
<exclude name="Squiz.Arrays.ArrayDeclaration.MultiLineNotAllowed"/><!-- even a single-value array can be written on multiple lines -->
173+
<exclude name="Squiz.Arrays.ArrayDeclaration.NoComma"/><!-- does not handle nested array access with complex keys on multiple lines; also already checked better by SlevomatCodingStandard.Arrays.TrailingArrayComma -->
174+
<exclude name="Squiz.Arrays.ArrayDeclaration.NoCommaAfterLast"/><!-- expects multi-value array always written on multiple lines -->
175+
<exclude name="Squiz.Arrays.ArrayDeclaration.SingleLineNotAllowed"/><!-- multiple values can be written on a single line -->
176+
<exclude name="Squiz.Arrays.ArrayDeclaration.ValueNoNewline"/><!-- false positives with nested arrays https://github.com/squizlabs/PHP_CodeSniffer/issues/2937#issuecomment-615262067 -->
177+
<exclude name="Squiz.Arrays.ArrayDeclaration.ValueNotAligned"/><!-- we don't want spacing with alignment -->
178+
</rule>
179+
<rule ref="Squiz.Classes.ClassFileName"/>
180+
<rule ref="Squiz.Classes.SelfMemberReference"/>
181+
<rule ref="Squiz.Classes.ValidClassName"/>
182+
<rule ref="Squiz.Commenting.DocCommentAlignment">
183+
<exclude name="Squiz.Commenting.DocCommentAlignment.SpaceAfterStar"/><!-- space needed for indented annotations -->
184+
</rule>
185+
<rule ref="Squiz.Commenting.EmptyCatchComment"/>
186+
<rule ref="Squiz.Commenting.FunctionComment">
187+
<exclude name="Squiz.Commenting.FunctionComment.EmptyThrows"/><!-- @throws are forbidden -->
188+
<exclude name="Squiz.Commenting.FunctionComment.IncorrectParamVarName"/><!-- requires long boolean and integer parameters -->
189+
<exclude name="Squiz.Commenting.FunctionComment.IncorrectTypeHint"/><!-- collection syntax such as string[] is not supported -->
190+
<exclude name="Squiz.Commenting.FunctionComment.InvalidReturn"/><!-- enforces incorrect types -->
191+
<exclude name="Squiz.Commenting.FunctionComment.InvalidReturnNotVoid"/><!-- is not able to detect return types such as string|null as correct -->
192+
<exclude name="Squiz.Commenting.FunctionComment.InvalidThrows"/><!-- @throws are forbidden -->
193+
<exclude name="Squiz.Commenting.FunctionComment.Missing"/><!-- PHPDoc is not required on all methods -->
194+
<exclude name="Squiz.Commenting.FunctionComment.MissingParamComment"/><!-- comments are not required for @param -->
195+
<exclude name="Squiz.Commenting.FunctionComment.MissingReturn"/><!-- void type is not used -->
196+
<exclude name="Squiz.Commenting.FunctionComment.ParamCommentFullStop"/><!-- comments don't have to be sentences -->
197+
<exclude name="Squiz.Commenting.FunctionComment.ParamCommentNotCapital"/><!-- comments don't have to be sentences -->
198+
<exclude name="Squiz.Commenting.FunctionComment.ScalarTypeHintMissing"/><!-- works only for code requiring PHP 7 code or better -->
199+
<exclude name="Squiz.Commenting.FunctionComment.SpacingAfterParamName"/><!-- we don't want spacing with alignment -->
200+
<exclude name="Squiz.Commenting.FunctionComment.SpacingAfterParamType"/><!-- we don't want spacing with alignment -->
201+
<exclude name="Squiz.Commenting.FunctionComment.ThrowsNoFullStop"/><!-- @throws are forbidden -->
202+
<exclude name="Squiz.Commenting.FunctionComment.ThrowsNotCapital"/><!-- @throws are forbidden -->
203+
<exclude name="Squiz.Commenting.FunctionComment.TypeHintMissing"/><!-- doesn't work with self as typehint -->
204+
</rule>
205+
<rule ref="Squiz.Commenting.FunctionComment.DuplicateReturn">
206+
<message>Only 1 @return annotation is allowed in a function comment</message>
207+
</rule>
208+
<rule ref="Squiz.Commenting.FunctionComment.ExtraParamComment">
209+
<message>Extra @param annotation</message>
210+
</rule>
211+
<rule ref="Squiz.Commenting.FunctionComment.InvalidNoReturn">
212+
<message>Function has no return statement, but annotation @return is present</message>
213+
</rule>
214+
<rule ref="Squiz.Commenting.FunctionComment.MissingParamTag">
215+
<message>@param annotation for parameter "%s" missing</message>
216+
</rule>
217+
<rule ref="Squiz.ControlStructures.ControlSignature"/>
218+
<rule ref="Squiz.ControlStructures.ForEachLoopDeclaration"/>
219+
<rule ref="Squiz.ControlStructures.ForLoopDeclaration"/>
220+
<rule ref="Squiz.Functions.FunctionDeclaration"/>
221+
<rule ref="Squiz.Functions.FunctionDeclarationArgumentSpacing">
222+
<properties>
223+
<property name="equalsSpacing" value="1"/>
224+
</properties>
225+
<exclude name="Squiz.Functions.FunctionDeclarationArgumentSpacing.SpacingAfterHint"/><!-- already checked by SlevomatCodingStandard.TypeHints.ParameterTypeHintSpacing.MultipleSpacesBetweenTypeHintAndParameter -->
226+
</rule>
227+
<rule ref="Squiz.Functions.GlobalFunction"/>
228+
<rule ref="Squiz.Functions.MultiLineFunctionDeclaration">
229+
<exclude name="Squiz.Functions.MultiLineFunctionDeclaration.NewlineBeforeOpenBrace"/><!-- clashes with OpeningFunctionBraceBsdAllman -->
230+
</rule>
231+
<rule ref="Squiz.Operators.IncrementDecrementUsage">
232+
<exclude name="Squiz.Operators.IncrementDecrementUsage.NoBrackets"/><!-- afaik there is no need for brackets -->
233+
</rule>
234+
<rule ref="Squiz.Operators.ValidLogicalOperators"/>
235+
<rule ref="Squiz.PHP.GlobalKeyword"/>
236+
<rule ref="Squiz.PHP.Heredoc"/>
237+
<rule ref="Squiz.PHP.InnerFunctions"/>
238+
<rule ref="Squiz.PHP.LowercasePHPFunctions"/>
239+
<rule ref="Squiz.PHP.NonExecutableCode"/>
240+
<rule ref="Squiz.Scope.MethodScope"/>
241+
<rule ref="Squiz.Scope.StaticThisUsage"/>
242+
<rule ref="Squiz.Strings.DoubleQuoteUsage"/>
243+
<rule ref="Squiz.Strings.DoubleQuoteUsage.ContainsVar">
244+
<message>Variable "%s" not allowed in double quoted string; use sprintf() instead</message>
245+
</rule>
246+
<rule ref="Squiz.Strings.ConcatenationSpacing">
247+
<properties>
248+
<property name="ignoreNewlines" value="true"/>
249+
<property name="spacing" value="1"/>
250+
</properties>
251+
</rule>
252+
<rule ref="Squiz.Strings.EchoedStrings"/>
253+
<rule ref="Squiz.WhiteSpace.CastSpacing"/>
254+
<rule ref="Squiz.WhiteSpace.FunctionOpeningBraceSpace"/>
255+
<rule ref="Squiz.WhiteSpace.FunctionSpacing">
256+
<exclude name="Squiz.WhiteSpace.FunctionSpacing.After"/><!-- does not allow PHPUnit ignore comments -->
257+
<exclude name="Squiz.WhiteSpace.FunctionSpacing.AfterLast"/><!-- does not allow PHPUnit ignore comments -->
258+
<properties>
259+
<property name="spacing" value="1"/>
260+
</properties>
261+
</rule>
262+
<rule ref="Squiz.WhiteSpace.LogicalOperatorSpacing"/>
263+
<rule ref="Squiz.WhiteSpace.MemberVarSpacing"/>
264+
<rule ref="Squiz.WhiteSpace.ObjectOperatorSpacing">
265+
<properties>
266+
<property name="ignoreNewlines" value="true"/>
267+
</properties>
268+
</rule>
269+
<rule ref="Squiz.WhiteSpace.OperatorSpacing">
270+
<properties>
271+
<property name="ignoreNewlines" value="true"/>
272+
<property name="ignoreSpacingBeforeAssignments" value="false"/>
273+
</properties>
274+
</rule>
275+
<rule ref="Squiz.WhiteSpace.ScopeClosingBrace"/>
276+
<rule ref="Squiz.WhiteSpace.ScopeKeywordSpacing"/>
277+
<rule ref="Squiz.WhiteSpace.SemicolonSpacing"/>
278+
<rule ref="Squiz.WhiteSpace.SuperfluousWhitespace"/>
28279
<rule ref="SlevomatCodingStandard.Arrays.DisallowImplicitArrayCreation" />
29280
<rule ref="SlevomatCodingStandard.Arrays.TrailingArrayComma" />
30281
<rule ref="SlevomatCodingStandard.Classes.ClassConstantVisibility" />
@@ -278,15 +529,9 @@
278529
<exclude-pattern>tests/unit/Api/Driver/Curl_exec_false_Mock.php</exclude-pattern>
279530
<exclude-pattern>tests/unit/Crypto/GlobalFunctionsMock.php</exclude-pattern>
280531
</rule>
281-
<rule ref="Consistence.NamingConventions.ValidVariableName.NotCamelCaps">
282-
<exclude-pattern>tests/unit/Crypto/GlobalFunctionsMock.php</exclude-pattern>
283-
</rule>
284532
<rule ref="Squiz.Functions.GlobalFunction.Found">
285533
<exclude-pattern>tests/SlevomatEET/Cryptography/OpenSslFunctionsMock.php</exclude-pattern>
286534
</rule>
287-
<rule ref="Consistence.NamingConventions.ValidVariableName.NotCamelCaps">
288-
<exclude-pattern>tests/SlevomatEET/Cryptography/OpenSslFunctionsMock.php</exclude-pattern>
289-
</rule>
290535
<rule ref="PSR1.Methods.CamelCapsMethodName.NotCamelCaps">
291536
<exclude-pattern>src/SoapClient.php</exclude-pattern>
292537
</rule>

0 commit comments

Comments
 (0)