Chore: Drop $loop usages in favor of Loop::get()#221
Chore: Drop $loop usages in favor of Loop::get()#221bpolaszek wants to merge 2 commits intoreactphp:3.xfrom
Conversation
|
Hey @bpolaszek, thank you for your pull request, always happy about contributions 👍 As I also wrote in reactphp/stream#176, changing the minimal supported PHP version to PHP 7.1 is something we want to see for ReactPHP v3, but I think we should use separate PRs for these topics to avoid bloating each pull request in complexity, which also makes them easier to review. In my opinion, we don't need the PHP 7.1 changes to remove the optional The rest of the changes are heading in a good direction. I think you can also take a look at clue/reactphp-redis#156 and use this as inspiration (for changes, PR description, commit history, etc.), as we also removed the optional Let's focus on either reactphp/stream#176 or this one here first to get a good feeling on what the PR should look like and then use this knowledge for the other pull requests. This way we can avoid doubling the work 👍 |
Still a few tests to fix.