[TASK] Add a fix Composer script to run all code fixers
#759
Conversation
This makes our life as developers a bit more convenient. Also run Rector before the style fixer as Rector is responsible for structural changes that need to be style-fixed after that.
oliverklee
added
enhancement
developer-specific
oliverklee
added this to the
9.0.0: Drop support for PHP < 7.2 and clean up things milestone
JakeQZ
left a comment
JakeQZ
left a comment
There was a problem hiding this comment.
Looks like we don't yet have Composer-Normalize on this project, but the previous ordering within fix:php I think was correct, and should not be changed.
| "@fix:php:rector", | ||
| "@fix:php:fixer" |
There was a problem hiding this comment.
F comes before R in the latin alphabet, so the existing ordering was correct.
There was a problem hiding this comment.
I changed this on purpose:
Also run Rector before the style fixer as Rector is responsible for structural changes that need to be style-fixed after that.
There was a problem hiding this comment.
I changed this on purpose:
Also run Rector before the style fixer as Rector is responsible for structural changes that need to be style-fixed after that.
Sorry, I missed that, and for some reason thought the script content itself would be reordered by Composer-Normalize :/
| "@fix:php:rector", | ||
| "@fix:php:fixer" |
There was a problem hiding this comment.
I changed this on purpose:
Also run Rector before the style fixer as Rector is responsible for structural changes that need to be style-fixed after that.
Sorry, I missed that, and for some reason thought the script content itself would be reordered by Composer-Normalize :/
This makes our life as developers a bit more convenient.
Also run Rector before the style fixer as Rector is responsible for structural changes that need to be style-fixed after that.