Enhancement: Normalize composer.json

php-http / message

HTTP Message related tools

http://php-http.org

MIT License

1.3k stars 42 forks source link

Enhancement: Normalize composer.json #115

Closed localheinz closed 5 years ago

localheinz commented 5 years ago

Q	A
Bug fix?	no
New feature?	no
BC breaks?	no
Deprecations?	no
Related tickets	n/a
Documentation	n/a
License	MIT

What's in this PR?

This PR

[x] requires localheinz/composer-normalize as development dependency
[x] runs composer validate and composer normalize --dry-run on Travis
[x] normalizes composer.json

💁‍♂️ For reference, see https://github.com/localheinz/composer-normalize.

localheinz commented 5 years ago

@dbu

I have made a few changes, thank you for suggesting to use localheinz/composer-normalize in the build pipeline!

localheinz commented 5 years ago

Hmm, Scrutinizer currently fails, see https://scrutinizer-ci.com/g/php-http/message/inspections/3d11bd3a-d6fc-459b-bdf4-1507e4c752ae:

dbu commented 5 years ago

does that mean that scrutinizer is using an outdated version of composer? can you check with the scrutinizer people? that said, i personally don't look at the scrutinizer output much - we could also drop it imho.

localheinz commented 5 years ago

Thank you, @dbu!