rearranges use statements even if file not touched

[matthbull]

its not a major thing, but it messes with VCS tracking quite a bit.

On files where no changes are made, the tool (which is excellent btw) changes the order of the use statements at the top of the file. This makes it look like you've touched the file from a VCS point of view which aggravates touchy tech-leads.

[umpirsky]

@matthbull Thanks for the feedback.

This is caused by 180d267, I added it to fix #13.

Until it's fixed you have 2 possible workarounds:

• Run php-cs-fixer fix --fixers=ordered_use on your code, then commit, then run upgrade fixer.

or

• Comment out 180d267#diff-76f67beab0c7251cf16ba13bf07c3ae1R43 and use patched version of the fixer.

[althaus]

Isn't it possible to check, if anything else is to fix, before running the OrderedUseFixer?

[umpirsky]

Yes, but that would require quite some changes.

[romaricdrigon]

+1, this is actually annoying. No way to make this configurable at runtime?
As with PHP CS Fixer, some rules can be excluded.

[romaricdrigon]

Done, I sent such a PR :)
In our case, from 130 modified files we go to only 30. This is a big step for readibility.