-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Mercure] update mercure documentation #15149
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tagged the new releases, we can merge this one now! I made some minor change requests.
In which version should we merge this? This article is available in 4.4, 5.2 and 5.x branches. Thanks! |
mercure and mercure-bundle don't follow symfony versioning, these changes apply to mercure 0.5 and mercure-bundle 0.3, so i'm not really sure. |
2ccaec5
to
271fc58
Compare
As symfony/mercure and mercure-bundle are still in 0.x versions, and because the latest versions support all maintained Symfony versions, I suggest to merge this in the 4.4 branch. |
@azjezz thanks for this contribution! And thanks to reviewers too! This is now merged in 4.4 branch and up. |
No description provided.