-
Notifications
You must be signed in to change notification settings - Fork 15
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
Php 8.0 eol and doesn’t work beyond #144
Comments
Interesting. I can def update this SDK to support 8.1+ and wasn't aware it was so broken as I haven't gotten any reports. I'm working on a major update (specifically for workflows), so I will see what I can backport to this version. |
What’s the eta? Workflows are the next thing I wanted to look at, for now trying to bump estate to 8.3 and need Kafka/service bus comms |
This would be great. I work with Mat, and we're hoping that Dapr will let us use a number of cloud services more easily in various PHP containers. The error I'm getting installing with
When I cloned the library itself to try updating dependencies and running tests,
Most of these seem like they can be fixed with Having said that, Haven't tried the integration tests. Hoping updating this won't be too much trouble 🤞 |
No ETA for this yet, but it is something I've been diligently working on because I want it too. Currently, I'm using a workflow library that I built before Dapr had it (https://github.com/bottledcode/durable-php) and I am working on replacing the runtime with Dapr instead of a bespoke implementation + NATS and combining it here. This will effectively make
I'd imagine all tests should pass. This part of the dapr api hasn't changed much (if at all). Updating the integration tests will be a pita, but doable.
Probably pretty easy. Those look like test dependencies and some stuff for if you aren't integrating with an existing project (i.e., using it in a 'standalone' mode). |
The durable stuff is interesting concept! |
So, just to keep you updated. The main issue with >8.1 is readonly, and with 8.4 including property hooks and aviz, this gets complicated when it comes to serialization: https://externals.io/message/125740 and https://externals.io/message/125772. Ie, how serialization works pre-8.4 and post-8.4 won't be compatible. I'm still working out the details but I'm going to make a jump directly for PHP 8.1 and 8.4 support, so there likely won't be a release until 8.4 is released in November as I will need to support both ways of handling asymmetric visibility and the 8.4 way is still being discussed to some degree. |
@withinboredom Any update on your progress? |
It's currently up-next on my todo list, once I finish my current project (this week). |
A bug was fixed up-stream in PHP 8.4.3 yesterday which will allow me to continue. On a roll! php/php-src#17101 |
Given that Php 8.0 is now eol, and this project doesn’t work with 8.1 is there anything happening with this project now? I saw a new maintainer was asked for, nothing happened and Kudos to @withinboredom for getting so far, but I don’t want to waste time and effort on dapr if this will be retired etc.
So what’s the plans for this Php SDK for dapr? Or do I look for another solution
The text was updated successfully, but these errors were encountered: