-
Notifications
You must be signed in to change notification settings - Fork 429
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
Support for AMPHP v3 #1064
Comments
Idd surely accept a PR for that. Currently, amphp/parallel (and parallel-functions) doesnt have a stable release yet - so we're kinda waiting on that. Not sure if it is possible to support both v2 and v3 at the same time, since the whole API of amp has changed. So, unless there are better ideas, I guess the best thing to do here is changing the API in grumphp v2 and let it use amp 3 Futures. |
Good news, an amp 3 compatible version of parallel has been released |
but parallel-functions does not have a compatible version yet |
So I've had this page open on my laptop for about 2 months now, meaning to say I'd be happy to help transition this package to use @stof You opening an issue kicked me to actually make a post. I'm not sure when I could promise any time, but I'd be happy to answer questions if someone more familiar with the library wants to tackle it first. |
Thanks for jumping on on the discussion @trowski. I'm planning to play around with amp v3 and GrumPHP for the rest of the day. |
@trowski actually, opening the issue on |
A small update: I think I covered all existing grumphp cases with amp v3 in the PR #1084. I was able to throw away a lot of strange quirks I needed to add to get things like stop-on-failure and parallel-functions to work in v2 of amp. It's nice to see how much the project has improved! Great stuff! Feel free to go through the PR and suggest improvements if I missed or misused anything :) |
Feel free to play around with the v2.x branch; |
Currently, grumphp requires AMPHP v2. It would be great if it could support both v2 and v3 to avoid blocking upgrades in projects using grumphp.
The text was updated successfully, but these errors were encountered: