Order imports alphabetically

[come-nc]

It seems phpstorm does not sort the use statements the same way as sort, and other editors may do yet something else.
So I propose to force the order in the coding standard so that the diff make sense and we do not argue about the order.

Signed-off-by: Côme Chilliet [email protected]

[come-nc]

I’m a bit sad to see that phpcs sides with phpstorm on this one and does not follow sort order.
But thankfully there is no option for changing that so we cannot argue about it.

What there is an option for is wether special kind of import should be sorted separately, I would vote for using it and not mix use function with use

[ChristophWurst]

I'm okay with this

[nickvergessen]

What there is an option for is wether special kind of import should be sorted separately, I would vote for using it and not mix use function with use

So like PSR12?
https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/master/doc/rules/import/ordered_imports.rst#rule-sets

Fine with that as well:

['imports_order' => ['class', 'function', 'const'], 'sort_algorithm' => 'alpha']

[come-nc]

What there is an option for is wether special kind of import should be sorted separately, I would vote for using it and not mix use function with use

So PSR12? https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/master/doc/rules/import/ordered_imports.rst#rule-sets

Fine with that as well.

No, alpha sorting order, PSR12 seems to be none.
But yeah order ['class', 'function', 'const'] like PSR12 is fine by me, I will push that.

[come-nc]

@ChristophWurst I suppose we should tag a new release for this to be deployed on the other repos?
Could you tag it?