From e5ce79b71af1012ea61fe990d01da8d94a372edc Mon Sep 17 00:00:00 2001 From: Bart Jaskulski <bjaskulski@protonmail.com> Date: Thu, 14 Nov 2024 08:59:37 +0100 Subject: [PATCH] feat: decrease severity for noisy sniffs Signed-off-by: Bart Jaskulski <bjaskulski@protonmail.com> --- WPDeskPlugin/ruleset.xml | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/WPDeskPlugin/ruleset.xml b/WPDeskPlugin/ruleset.xml index 97e9033..012079e 100644 --- a/WPDeskPlugin/ruleset.xml +++ b/WPDeskPlugin/ruleset.xml @@ -45,6 +45,24 @@ <!-- Since WPCS 3.0 exceptions are treated as output and requires escaping. We don't want that... --> <exclude name="WordPress.Security.EscapeOutput.ExceptionNotEscaped" /> + + <!-- $default parameter sometimes just make sense. --> + <exclude name="Universal.NamingConventions.NoReservedKeywordParameterNames"/> + + <!-- Complex ternary are OK. --> + <exclude name="Squiz.ControlStructures.InlineIfDeclaration.NotSingleLine" /> + <exclude name="Squiz.ControlStructures.InlineIfDeclaration.NoBrackets"/> + </rule> + + <!-- Those below are mostly noise, but worth to keep an eye on them. --> + <rule ref="Generic.CodeAnalysis.EmptyStatement.DetectedCatch"> + <type>warning</type> + </rule> + <rule ref="WordPress.NamingConventions"> + <type>warning</type> + </rule> + <rule ref="WordPress.WP.I18n.MissingTranslatorsComment"> + <type>warning</type> </rule> <!-- Allow slash in hook name. --> -- GitLab