Skip to content

Commit

Permalink
[TASK] Improve readability of php-cs-fixer configuration (#1553)
Browse files Browse the repository at this point in the history
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`.
  • Loading branch information
d-s-codappix authored Jan 16, 2025
1 parent 8b10a0b commit 0dcc435
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .php-cs-fixer.php
Original file line number Diff line number Diff line change
Expand Up @@ -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());
Expand Down

0 comments on commit 0dcc435

Please sign in to comment.