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 react/promise 3 #33

Merged
merged 1 commit into from
Sep 11, 2022

Conversation

WyriHaximus
Copy link
Member

No description provided.

@clue clue added this to the v1.9.0 milestone Sep 9, 2022
@clue
Copy link
Member

clue commented Sep 9, 2022

@WyriHaximus Good initiative and would love to see this getting some traction again! What are your plans for this PR? It looks like all outstanding PRs have been merged in the meantime? (reactphp/dns#153, reactphp/promise-timer#55, reactphp/promise-timer#54, reactphp/promise-timer#37)

@WyriHaximus WyriHaximus force-pushed the promise-3 branch 2 times, most recently from d45b662 to 98ab0fd Compare September 9, 2022 18:09
@WyriHaximus WyriHaximus changed the title [WIP] Forward compatibility with react/promise 3 Forward compatibility with react/promise 3 Sep 9, 2022
@WyriHaximus WyriHaximus requested a review from clue September 9, 2022 18:10
@WyriHaximus
Copy link
Member Author

@clue https://github.com/reactphp/datagram/actions/runs/3024442084 shows it passing with promise v3 (@dev) working for supported PHP versions.

@WyriHaximus WyriHaximus force-pushed the promise-3 branch 2 times, most recently from ba04c8c to 1635cc1 Compare September 9, 2022 18:12
src/Factory.php Outdated Show resolved Hide resolved
Copy link
Member

@clue clue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@WyriHaximus Thanks for the update, changes LGTM! :shipit:

@clue clue merged commit a85418a into reactphp:1.x Sep 11, 2022
@WyriHaximus WyriHaximus deleted the promise-3 branch September 11, 2022 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants