Hey@bpolaszek, thank you for your pull request, always happy about contributions 👍

As I also wrote inreactphp/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$loop parameter here.

The rest of the changes are heading in a good direction. I think you can also take a look atclue/reactphp-redis#156 and use this as inspiration (for changes, PR description, commit history, etc.), as we also removed the optional$loop parameter in there. Thanks to our consistency across all our projects, we can apply similar changes in here.

Let's focus on eitherreactphp/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 👍