Closed ghost closed 1 year ago
I have been working my way through the php 8 updates of the consolidation components, but only got partway through and currently do not have a lot of time for open source work. This one will probably just work. PRs welcome.
Who (else) can review PR in #41 ?
I'll merge with any +1 from the community verifying correct behavior on php8
As reported on https://github.com/dpi/dogit/issues/6#issue-953179390, the project seems to work just fine by using ignore-platform-reqs. So it seems these minor composer.json changes in the PR are unlikely to break things.
It seems like the project is indeed working with PHP8. Is there anything we could specifically check that could not be working?
For people who want to install without this PR merged yet:
composer global require --ignore-platform-req php consolidation/cgr
For your reference finally I give up with cgr, but shift to OBS based (https://build.opensuse.org) deb/rpm management, e.g.:
composer/composer
in deb/rpm (see https://github.com/alvistack/composer-composer/commit/7866c2ca493a1d13bf189b1ed833aad3e873d865 commit message)/usr/share/php/vendor
(see https://github.com/alvistack/composer-composer/commit/7866c2ca493a1d13bf189b1ed833aad3e873d865#diff-889c5495b68d2a0c0d1f238818716a71939918499efbc0fc6d2994319624fc67R7-R17)/usr/share/php/vendor/composer/bin/composer composer
to /usr/bin/composer
Some other else example:
I'll merge with any +1 from the community verifying correct behavior on php8
Cgr works with PHP 8.1.
No one put the +1 on the PR, looks like I missed over a year of +1's over here on the issue. Not being pedantic, I just get a lot of GitHub notifications and didn't notice. Going to go merge the PR now.
This appears to be merged but not published. Do we need to install 2.x-dev instead of the latest which is 2.0.5?
Forgot to tag a release. Tagged 2.0.6.
perfect, thank you!
Hello friend,
Have you plans to accept php 8 in requirements?