search

Comcast / php-legal-licenses

A utility to help generate a file containing information about dependencies including the full license text.
https://packagist.org/packages/comcast/php-legal-licenses
Apache License 2.0
75 stars 25 forks source link

Support PHP 8.0.0 #18

Closed jonathan-shahen closed 3 years ago

jonathan-shahen commented 3 years ago

Is your feature request related to a problem? Please describe. I am unable to use your software with PHP 8

Describe the solution you'd like Update composer.json to allow ^8.0

$ composer update
Loading composer repositories with package information
Updating dependencies                                 
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - comcast/php-legal-licenses[v1.1.0, ..., v1.1.2] require php ^5.6 || ^7.0 || ^7.1 -> your php version (8.0.0) does not satisfy that requirement.
    - comcast/php-legal-licenses[v1.1.3, ..., v1.1.4] require php ^5.6 || ^7.1 || ^7.2 || ^7.3 || ^7.4 -> your php version (8.0.0) does not satisfy that requirement.
    - Root composer.json requires comcast/php-legal-licenses ^1.1 -> satisfiable by comcast/php-legal-licenses[v1.1.0, ..., v1.1.4].
judge2020 commented 3 years ago

Yep - this is the last dependency my project depends on that doesn't indicate PHP 8 compatibility. Installing with --ignore-platform-reqs then vendor/bin/php-legal-licenses generate seems to work in my repo on PHP 8.

orediggerco commented 3 years ago

Thanks for submitting this, I'll get it updated soon!

jonathan-shahen commented 3 years ago

@orediggerco I submitted a PR

orediggerco commented 3 years ago

PHP Version constraints are now removed as of v1.1.5.