Skip to content
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

Forward compatibility with upcoming Promise v3 #35

Merged
merged 2 commits into from
Sep 2, 2022

Conversation

clue
Copy link
Owner

@clue clue commented Jun 22, 2022

@clue clue added the new feature New feature or request label Jun 22, 2022
@clue clue added this to the v1.4.0 milestone Jun 22, 2022
@clue clue changed the title Forward compatibility with upcoming Promise v3 [WIP] Forward compatibility with upcoming Promise v3 Jun 22, 2022
@clue
Copy link
Owner Author

clue commented Jun 22, 2022

The first commit updates this to the currently unreleased Socket component to show how this only depends on reactphp/socket#214, clue/reactphp-socks#106 and friends-of-reactphp/mysql#157 (the build should be green). The second commit updates this to the releases that have yet to be tagged. This is expected to fail at the moment and should be green once the releases are tagged and the build is restarted.

@SimonFrings
Copy link
Contributor

The HHVM run will currently fail due to composers new version. #36 will fix this issue by downgrading composer for the hhvm build.

@clue clue changed the title [WIP] Forward compatibility with upcoming Promise v3 Forward compatibility with upcoming Promise v3 Sep 2, 2022
@clue
Copy link
Owner Author

clue commented Sep 2, 2022

Updated now that Socket v1.12.0 and SOCKS v1.4.0 have been released, this is now ready for review :shipit:

(Unrelated HHVM build error has been fixed via #36 in the meantime)

@clue clue requested a review from SimonFrings September 2, 2022 11:07
@SimonFrings SimonFrings merged commit 3a523ad into clue:main Sep 2, 2022
@clue clue deleted the promise-v3 branch September 2, 2022 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants