dephpend
dephpend copied to clipboard
Upgrade to PHP8.1
This PR upgrades the entire package and its dependencies to rely on php 8.1
.
The main drive for the php update is to be able to use all latest versions of the existing packages.
Special mention
This specific commit prevents PHP Enum classes from breaking.
It used to break saying that $node->isAbstract()
doesn't exist for Node\Stmt\Enum_
.
This PR includes
- Added github actions to include tests
- Updated PHPUnit assertions to work with phpunit 9.5
- Updated php-cs-fixer --rules label. And updated the codebase afterwards with the new psr12 standard.
-
$method->getReturnType()->getName()
update. - The Dockerfile is also updated to use PHP8.1 too.
There are some failing tests that I can't make it work with github actions:
- Install
json.so
extension: I addedjson
to the list of extensions, but it didn't seem to work. - I added a new test for Enums, and now there is a new error for the test. I couldn't figure out how the tests well too well, but I have a suspicion that the Enums are not printing anything. I run this updated version against my codebase and it was able to generate the Text version correctly.
I opened it as a draft to get feedback and assistance before committing to fix the tests. Thanks in advance for this package and for the help!
Hey, thank you so much for this PR and sorry for letting you wait this long. Work's been very busy, but I'm getting on it right now :)
@mihaeu what is the progress here?
@mihaeu please? 1.5 years for a PR!
@mihaeu Necesitas ayuda? Need help? would love to see this get pushed through
@juampi92 @mihaeu any update on this PR or PHP 8.1 support
@juampi92 @mihaeu Thanks for merging! Could you please release this?