From 0a8024e9174b5b4f7cd4f7846be8b0cb365b2773 Mon Sep 17 00:00:00 2001 From: jrfnl Date: Sun, 7 Apr 2024 17:34:39 +0200 Subject: [PATCH] YoastCS: always flag comma spacing in function calls The `Universal.WhiteSpace.CommaSpacing` has modular error codes to allow for preventing duplicate error messages about the same issue from different sniffs. WordPressCS excludes the "function call" related error codes from the sniff to prevent an overlap with the `Generic.Functions.FunctionCallArgumentSpacing` sniff, but this doesn't play nice with a code base using short arrays as the spacing after commas in a short array being passed as a parameter in a function call will then also be skipped. This change prevents this issue and ensures that the spacing around those commas will be checked correctly. --- Yoast/ruleset.xml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/Yoast/ruleset.xml b/Yoast/ruleset.xml index c28bf55..1de3b22 100644 --- a/Yoast/ruleset.xml +++ b/Yoast/ruleset.xml @@ -147,6 +147,19 @@ + + + 5 + + + 5 + + + 5 + +