Skip to content

Conversation

@liviucmg
Copy link

Fixes issue #969.

@mbabker
Copy link
Contributor

mbabker commented Jul 4, 2022

This isn't compatible with PHP 5, which the 0.4 branch still supports.

@damoclark
Copy link

Hey @mbabker

Just found my way to this issue after upgrading my Symfony 5.4 to 6.1 with gos/web-socket-bundle. I took a peek at the 0.5 branch, but it still has PHP 5 as supported in its composer.json too.

Is the plan for Ratchet 0.5 to drop PHP 5 and pick up PHP 8.1? Is there a way to progress this forward so that Ratchet can be used with Symfony 6.1?

Many thanks.

Damo.

@PaulRotmann
Copy link
Contributor

Hey! Thanks for this pull request 🙏

As part of our cleanup initiative (#1100), we're closing old PRs that have already been resolved. We've recently merged #1095 which implements the same idea, but turned out to be a bit more involved than initially anticipated.

If you have further improvements to suggest, feel free to open a new pull request. We appreciate your contributions and welcome any future enhancements. If you want to support Ratchet, please consider sponsoring our work! ❤️

Thanks again for your contribution.

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.

5 participants