Greg Korba comments

Results 540 comments of


                                            Greg Korba

`binary_operator_spaces` problem with several shifts in single line

Reopening because #7593 was reverted.

Suggest people use the JIT on PHP 8+

Since it is not a core problem, but rather optimisation thing (that is out of the scope of the tool itself), and because it did not gain traction for 2...

OrderedImportsFixer: Alpha sorting so paths are alpha-sorted-grouped and classes are alpha-sorted

Hi @garrettboone, thank you for your suggestion. Personally I don't find this kind of ordering useful because IMHO it's not intuitive. For me it does not matter how deep classes...

NullableTypeDeclarationForDefaultNullValue - remove `|null` from union types when null is the default

@HypeMC any chance for a rebase? Also, I believe that it would be great to have an option for choosing nullable syntax applied to single type (related to #7002), otherwise...

NullableTypeDeclarationForDefaultNullValue - remove `|null` from union types when null is the default

Thank you very much @HypeMC, great work 🍻!

NoExtraBlankLinesFixer and curly braces

FYI:

Incorrect PSR-12 brace placement for anonymous classes

Hey! Thanks for your report. We identified that `braces` fixer (that is applied in your example) is too complex and has too many responsibilities. With that, [we decided to deprecate...

Describe your own config

I believe #3047 could solve this, but it requires some more work.

Describe your own config

FYI: works ongoing in #7254.

Describe your own config

@metalmini can you take a look at #7254 (e.g. [here](https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/pull/7254#issuecomment-1701981669)) and tell if such command would fit your needs?