KnpLabs / packagist-api

PHP API for Packagist
MIT License
182 stars 46 forks source link

Update to require PHP 7.4 as the minimum version, add strict typing #64

Closed robbieaverill closed 3 years ago

robbieaverill commented 4 years ago

This is a breaking change, so will require a 2.0 release of this library. I'll leave this pull request open for a week or two to gather any feedback that may come in before merging.

robbieaverill commented 4 years ago

It is probably worth including PHP 8 support before we ship this new major version.

garak commented 4 years ago

It is probably worth including PHP 8 support before we ship this new major version.

Just add it to travis and README

GrahamCampbell commented 3 years ago

https://github.com/KnpLabs/packagist-api/pull/68 is less radical, and addresses the urgent issues at hand: support for Guzzle 7 and PHP 8.

GrahamCampbell commented 3 years ago

After that PR is merged, this one can be rebased.

robbieaverill commented 3 years ago

Merged by hand as part of #67