From 0dcc435942d47c7e4329254a267f9a8e975fc047 Mon Sep 17 00:00:00 2001 From: Daniel Siepmann Date: Thu, 16 Jan 2025 09:03:43 +0100 Subject: [PATCH] [TASK] Improve readability of php-cs-fixer configuration (#1553) We now use `addRules()` method instead of fiddling around with an array of rules. It allows us to overwrite existing rules, or even disable existing rules by setting them to `false`. --- .php-cs-fixer.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/.php-cs-fixer.php b/.php-cs-fixer.php index 28aec1bd..8281bf84 100644 --- a/.php-cs-fixer.php +++ b/.php-cs-fixer.php @@ -5,11 +5,11 @@ $config = CsFixerConfig::create(); -// This is required as long as we are on PHPUnit 9.x. It can be removed after the switch to PHPUnit 10.x. -// @see https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/issues/8337 -$rules = $config->getRules(); -$rules['php_unit_test_case_static_method_calls'] = ['call_type' => 'self', 'methods' => ['createStub' => 'this']]; -$config->setRules($rules); +$config->addRules([ + // This is required as long as we are on PHPUnit 9.x. It can be removed after the switch to PHPUnit 10.x. + // @see https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/issues/8337 + 'php_unit_test_case_static_method_calls' => ['call_type' => 'self', 'methods' => ['createStub' => 'this']], +]); // @TODO 4.0 no need to call this manually $config->setParallelConfig(ParallelConfigFactory::detect());