Dieter Beck
Dieter Beck
@Arhell Friendly ping :) Can this be merged? It would be great to have the usage of named arguments documented.
This PR can be closed if https://github.com/Codeception/Codeception/pull/6812 is accepted because it adjusts the implementation to the documented behavior.
Seems like the PHP 8.4 pipeline is blocked by https://github.com/laminas/laminas-stdlib/pull/118
@LOTBAegurpegui Can you share any error logs (webserver, PHP)? Alternatively, please share the code which causes the error.
> I made a workaround adding 8 times issets @C1977 What are the code places you had to adjust? Do you plan to create a PR?
It's purely a reminder. I haven't done any research yet.
https://github.com/jumbojett/OpenID-Connect-PHP/pull/462 has already been merged. Are there any plans to tag a new release to get rid of the PHP 8.4 deprecation warnings?
I created https://github.com/Codeception/Codeception/pull/6812 which is based on https://github.com/Codeception/Codeception/pull/6752 which was about the same issue. This PR can be closed if my PR is accepted.
Is the "reporters" option added by phpStorm? According to the changelog it is not supported in v5 anymore https://github.com/Codeception/Codeception/blob/6e06224627dcd89e7d4753f44ba4df35034b6314/CHANGELOG-5.x.md?plain=1#L182
@francislavoie I wonder if the tests could be adjusted to verify that the new logic works as expected. This can be achieved by adjusting the expected exception messages in https://github.com/francislavoie/module-rest/blob/abc599a9992db4ed2d6f6165a6b3ad8fe901ae89/tests/unit/Codeception/Module/RestTest.php#L715....