New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: handle native enum casts #1494
Conversation
cd499c8
to
2731db8
Compare
Weird, how did this pass in the CI with PHP 8.0? 🤔 Anyway, can you also add a test with backed enums? |
What's the status of this PR? Is there something missing that I can help to provide? The bug fixed by this PR is still causing a bunch of false positives on my side, which is quite annoying, so: how can I help accelerate it to get it merged & released? |
2731db8
to
30f30c4
Compare
@erikgaal Could it be that |
@szepeviktor Yes, that's completely correct. It's not using the native PHP reflection (which would crash on 8.0), but the |
This looks good. I'm merging it now, and gonna add some more test cases for the writeable type later. Thank you. |
@canvural Do you have an estimation for when a release will be made with this included? |
Extends my previous PR, where I forgot to implement support for native enum casts.
Changes
Breaking changes