Again: shouldn't dist config be changed? sweat_smile

Reading the comment in the dist confing probably better would be to disable modernize_strpos for @PhpCsFixer:risky (until we stop supporting PHP < 8).

Thanks @kubawerlos 🍻

In terms of change use_nullable_type_declaration default, I just wanted to mention using implicit type rather than explicit seems to not be recommended by PHP

As of PHP 8.0.0, declaring mandatory arguments after optional arguments is deprecated. This can generally be resolved by dropping the default value, since it will never be used. One exception to this rule are arguments of the form Type $param = null, where the null default makes the type implicitly nullable. This usage remains allowed, though it is recommended to use an explicit nullable type instead.

https://www.php.net/manual/en/functions.arguments.php

https://www.php.net/manual/en/language.types.declarations.php#language.types.declarations.nullable

@darkin1 the part:

using implicit type rather than explicit seems to not be recommended

for me applies to pretty much everything, it was added not because it is good practice, but because it is in current Symofny's standards.