sveneld commented on PR #3792:
URL: https://github.com/apache/thrift/pull/3792#issuecomment-5551625587
Since this PR already touches `composer.json`, could you also add
`"sort-packages": true` to the `config` block?
```json
"config": {
"sort-packages": true,
"allow-plugins": {
"php-http/discovery": true
}
}
```
This makes Composer keep `require`/`require-dev` alphabetically sorted
automatically on future `composer require`/`remove`, matching the ordering this
PR already applied by hand — so a reformat-only PR like this one shouldn't be
needed again. No extra validator/CI step is being requested for this —
`composer.json` changes rarely enough that it can stay a manual review point.
Also, could you squash the two commits into one before merge, per the
[contributing
guide](https://github.com/apache/thrift/blob/master/CONTRIBUTING.md)?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
