Skip to content

Conversation

@clue
Copy link
Member

@clueclue commented Jun 22, 2022

@WyriHaximusWyriHaximus added this to the v1.8.0 milestone Aug 22, 2022
@clue
Copy link
MemberAuthor

clue commented Aug 24, 2022

Rebased to fix merge conflict now that #464 has been merged. There are still a couple of failing tests that look like some unexpected behavior changes in Promise v3 as also noticed by @WyriHaximus in reactphp/socket#214 (comment). Considering the HTTP component depends on the Socket component, let's continue this discussion in the upstream PR. I'll make sure to update this PR once we make progress.

@clue
Copy link
MemberAuthor

clue commented Aug 26, 2022

Updated to use Socket v1.12.0 and apply the same changes to fix a race in the tests as discussed in reactphp/socket#214 (comment), so the build should now be green. This is still WIP as this currently still depends on updated dev dependencies that I'll try to get shipped no later than next week.

@clueclueforce-pushed the promise-v3 branch 2 times, most recently from a0399bd to 78cfb44CompareSeptember 2, 2022 14:39
@clueclue changed the title [WIP] Forward compatibility with upcoming Promise v3Forward compatibility with upcoming Promise v3Sep 2, 2022
@clue
Copy link
MemberAuthor

clue commented Sep 2, 2022

Updated to stable releases now that all dependencies have been released:

Also worked around another nasty HHVM build error because legacy HHVM does not support the iterable type in Promise v3 (reactphp/promise#225).

This is now ready for review :shipit:

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@clue@WyriHaximus