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

Add support for react/promise v3 #217

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

WyriHaximus
Copy link
Contributor

With v3 come a whole list of changes including end of promise chain detection, the removal of the CancellablePromiseInterface interface, and type templating.

@WyriHaximus
Copy link
Contributor Author

@davidwdan && @mbonneau I've also included the changes for type templates into this PR, do you prefer to have it in one big PR or split it into two?

@davidwdan
Copy link
Member

@WyriHaximus I think doing that in a separate PR would make sense. I'm very excited about template support.

@WyriHaximus WyriHaximus force-pushed the promise-v3 branch 2 times, most recently from 7dec0f8 to 1b94c3f Compare August 7, 2023 18:00
@WyriHaximus WyriHaximus changed the title [WIP] Add support for react/promise v3 Add support for react/promise v3 Aug 7, 2023
@WyriHaximus WyriHaximus marked this pull request as ready for review August 7, 2023 18:02
With v3 come a whole list of [changes](https://github.com/reactphp/promise/releases/tag/v3.0.0)
including end of promise chain detection, the removal of the `CancellablePromiseInterface`
interface, and type templating.
@WyriHaximus
Copy link
Contributor Author

@davidwdan Cool, moved that to #218 making this PR ready for review 🚀 .

@davidwdan davidwdan merged commit 83322e8 into ReactiveX:master Nov 27, 2023
12 checks passed
@WyriHaximus WyriHaximus deleted the promise-v3 branch November 27, 2023 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants