discord-php / DiscordPHP

An API to interact with the popular messaging app Discord
MIT License
977 stars 236 forks source link

Promises v3 Checklist #1149

Open valzargaming opened 1 year ago

valzargaming commented 1 year ago

Per https://github.com/reactphp/promise/releases/tag/v3.0.0

Irrelevant:

Breaking changes:

Non-breaking changes:

key2peace commented 1 year ago

All this should be in https://github.com/discord-php/DiscordPHP/tree/promise-v3 now

valzargaming commented 1 year ago

ExtendedPromiseInterface extends PromiseInterface, and Promise v2 implements ExtendedPromiseInterface, so instead of replacing the existing typehinting to PromiseInterface we could just use React\Promise\Promise, which would overcome the hurdle and be better for futureproofing code. This seems to be the ReactPHP developers' intent as well, given they have made all classes final to encourage composition.

shehi commented 1 year ago

It's painful to read this @valzargaming :) Code-ify more please.

valzargaming commented 1 year ago

It's painful to read this @valzargaming :) Code-ify more please.

This is already coded in the promises v3 branch as of today.