Skip to content

Conversation

@weaverryan
Copy link
Member

See symfony/symfony#31872

But also the conversation on symfony/messenger@47d2b91

So, I'm not actually sure if that commit is correct and the docs should be updated, or if that commit is wrong. Opening this up to at least have some placeholder to figure that out.

@javiereguiluz
Copy link
Member

The code PR was merged, so let's merge this too. Thanks Ryan.

@javiereguiluz javiereguiluz merged commit c76afd3 into symfony:4.3 Jun 13, 2019
javiereguiluz added a commit that referenced this pull request Jun 13, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

Redis 4.3 is required

See symfony/symfony#31872

But also the conversation on symfony/messenger@47d2b91

So, I'm not actually sure if that commit is correct and the docs should be updated, or if that commit is wrong. Opening this up to at least have some placeholder to figure that out.

Commits
-------

c76afd3 Redis 4.3 is required
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.

3 participants