diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2025-09-22 22:02:59 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2025-09-22 22:02:59 +0200 |
| commit | 833007f5860deae232bacf321d268124123d6749 (patch) | |
| tree | be6e18023f5e127545ab8cc1ef743a3a7e0c12ae /phpcs.xml | |
| parent | aadbdabd8329d7dec43f71e15e3433fa8fe5e4d4 (diff) | |
Upgrade to PHP_CodeSniffer 4 (#7993)
https://github.com/PHPCSStandards/PHP_CodeSniffer/wiki/Version-4.0-User-Upgrade-Guide
A couple of rules have changed name. Exclusion paths have slightly changed behaviour.
Main change is that it was unhappy with our i18n header comments, so minor edit there, since that rule kind of makes sense the rest of the time
Diffstat (limited to 'phpcs.xml')
| -rw-r--r-- | phpcs.xml | 40 |
1 files changed, 20 insertions, 20 deletions
@@ -2,24 +2,23 @@ <ruleset name="FreshRSS"> <arg name="extensions" value="php,phtml"/> <arg name="tab-width" value="4"/> - <exclude-pattern>./.git/</exclude-pattern> - <exclude-pattern>./data/config.php</exclude-pattern> - <exclude-pattern>./data/update.php</exclude-pattern> - <exclude-pattern>./data/users/*/config.php</exclude-pattern> - <exclude-pattern>./extensions/</exclude-pattern> - <exclude-pattern>./lib/http-conditional.php</exclude-pattern> - <exclude-pattern>./lib/marienfressinaud/</exclude-pattern> - <exclude-pattern>./lib/phpgt/</exclude-pattern> - <exclude-pattern>./lib/phpmailer/</exclude-pattern> - <exclude-pattern>./lib/simplepie/</exclude-pattern> - <exclude-pattern>./node_modules/</exclude-pattern> - <exclude-pattern>./p/scripts/vendor/</exclude-pattern> - <exclude-pattern>./vendor/</exclude-pattern> + <exclude-pattern>/\.git/*</exclude-pattern> + <exclude-pattern>/data/config.php</exclude-pattern> + <exclude-pattern>/data/update.php</exclude-pattern> + <exclude-pattern>/data/users/*/config.php</exclude-pattern> + <exclude-pattern>/(?-i:extensions)/*</exclude-pattern><!-- Case sensitive to allow our Extensions repo --> + <exclude-pattern>/lib/http-conditional.php</exclude-pattern> + <exclude-pattern>/lib/marienfressinaud/</exclude-pattern> + <exclude-pattern>/lib/phpgt/*</exclude-pattern> + <exclude-pattern>/lib/phpmailer/*</exclude-pattern> + <exclude-pattern>/lib/simplepie/*</exclude-pattern> + <exclude-pattern>/node_modules/*</exclude-pattern> + <exclude-pattern>/p/scripts/vendor/*</exclude-pattern> + <exclude-pattern>/vendor/*</exclude-pattern> <!-- Additional exclusions for Extensions: --> - <exclude-pattern>./symbolic/</exclude-pattern> - <exclude-pattern>./third-party/</exclude-pattern> - <exclude-pattern>./tmp/</exclude-pattern> - <exclude-pattern>./xExtension-TTRSS_API</exclude-pattern> + <exclude-pattern>/symbolic/*</exclude-pattern> + <exclude-pattern>/third-party/*</exclude-pattern> + <exclude-pattern>/tmp/*</exclude-pattern> <rule ref="PSR12"> <exclude name="Generic.ControlStructures.InlineControlStructure.NotAllowed"/> <exclude name="Generic.Formatting.DisallowMultipleStatements.SameLine"/> @@ -34,7 +33,8 @@ <exclude name="PSR12.ControlStructures.ControlStructureSpacing.CloseParenthesisLine"/> <exclude name="PSR12.ControlStructures.ControlStructureSpacing.FirstExpressionLine"/><!-- Consider using PSR12 defaults instead --> <exclude name="PSR12.Files.FileHeader.IncorrectOrder"/><!-- Consider using PSR12 defaults instead --> - <exclude name="PSR12.Files.FileHeader.SpacingAfterBlock"/><!-- Consider using PSR12 defaults instead --> + <exclude name="PSR12.Files.FileHeader.SpacingAfterDeclareBlock"/><!-- Consider using PSR12 defaults instead --> + <exclude name="PSR12.Files.FileHeader.SpacingAfterTagBlock"/><!-- Consider using PSR12 defaults instead --> <exclude name="PSR12.Traits.UseDeclaration.MultipleImport"/> <exclude name="PSR2.Classes.ClassDeclaration.CloseBraceAfterBody"/><!-- Consider using PSR12 defaults instead --> <exclude name="PSR2.Classes.ClassDeclaration.OpenBraceNewLine"/><!-- Consider using PSR12 defaults instead --> @@ -46,7 +46,7 @@ <exclude name="PSR2.Methods.FunctionCallSignature.Indent"/> <exclude name="PSR2.Methods.FunctionCallSignature.MultipleArguments"/> <exclude name="PSR2.Methods.MethodDeclaration.Underscore"/> - <exclude name="Squiz.Classes.ValidClassName.NotCamelCaps"/> + <exclude name="Squiz.Classes.ValidClassName.NotPascalCase"/> <exclude name="Squiz.Functions.MultiLineFunctionDeclaration.BraceOnSameLine"/> <exclude name="Squiz.Functions.MultiLineFunctionDeclaration.CloseBracketLine"/> <exclude name="Squiz.Functions.MultiLineFunctionDeclaration.ContentAfterBrace"/> @@ -65,7 +65,7 @@ <property name="lineLimit" value="165"/> <property name="absoluteLineLimit" value="190"/> </properties> - <exclude-pattern>/app/i18n/*\.php$</exclude-pattern> + <exclude-pattern>/i18n/*\.php$</exclude-pattern> <exclude-pattern>*\.phtml$</exclude-pattern> </rule> <rule ref="Generic.Functions.OpeningFunctionBraceKernighanRitchie"/><!-- Consider using PSR12 defaults instead --> |