-
-
Notifications
You must be signed in to change notification settings - Fork 240
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sort packages in patches section following sort-packages config #341
Comments
Some notes:
|
This comment has been minimized.
This comment has been minimized.
Probably not going to do this. The order that patches are applied to a dependency is not something that can just be changed in most cases. |
@cweagans @Pasqualle – This issue is not about the order of patches but about the order of packages in the patches section. Same as the order of packages in the require section. In a project with lots of patches having to adjust this section, alphabetically ordered packages help a lot. |
Yes, I agree, packages should be sorted automatically, same as composer does it. Please re-open as the rejection reason is invalid. Packages can (and should) be sorted. |
@JPustkuchen – Can you maybe rephrase the title? Something like "Sort packages in patches section following sort-packages config" or similar? |
I don't have time to work on this, but PRs are welcome! |
Problem:
If you have to handle a lot of patches in large projects or even have to update / diff them on a regular basis, a logical order of the packages patched helps a lot. Currently, the user has to sort the patches by package name himself.
Proposed solution:
It would be very handy to have an option to automatically update the patches order by package name like
for packages.
Great work, your plugin is one of the very most important for composer ever!
The text was updated successfully, but these errors were encountered: